commit adc6103aa04f59a7046c2df37f2491766927f8f6
author Tiger Huang <tigerhuang@google.com> Thu Dec 24 20:40:22 2020 +0800
committer Tiger Huang <tigerhuang@google.com> Thu Dec 24 20:40:22 2020 +0800
tree 98693d68ed1fee8cba284a7ce1a07b75438704ca
parent 2efdb860dc9dc0736bc01bc76f4d44ec2acb9e2d

Use LIGHT_BG_COLOR as the system bar color in the tests

Previously, we choosed a darker system-bar-background color which would
produce many unexpected hue pixels due to anti-aliasing. This CL uses
the same background color as other tests. The color will be closer to
the foreground/icon color, which will produce fewer unexpected hue
pixels.

Fix: 176114625
Test: atest LightBarTests
Change-Id: Icf387430f0722ba1dfb064648d223e5a1c6a3bf0

tests/tests/systemui/src/android/systemui/cts/LightBarTests.java

diff --git a/tests/tests/systemui/src/android/systemui/cts/LightBarTests.java b/tests/tests/systemui/src/android/systemui/cts/LightBarTests.java
index 5a08fe2..8fbd429 100644
--- a/tests/tests/systemui/src/android/systemui/cts/LightBarTests.java
+++ b/tests/tests/systemui/src/android/systemui/cts/LightBarTests.java
@@ -100,11 +100,10 @@
         assumeHasColoredStatusBar(mActivityRule);
 
         runInNotificationSession(() -> {
-            final int darkBackground = Color.rgb(128, 0, 0);
             final LightBarActivity activity = mActivityRule.getActivity();
             activity.runOnUiThread(() -> {
-                activity.getWindow().setStatusBarColor(darkBackground);
-                activity.getWindow().setNavigationBarColor(darkBackground);
+                activity.getWindow().setStatusBarColor(LIGHT_BG_COLOR);
+                activity.getWindow().setNavigationBarColor(LIGHT_BG_COLOR);
 
                 activity.setLightStatusBarLegacy(true);
                 activity.setLightNavigationBarLegacy(true);
@@ -117,7 +116,7 @@
             Thread.sleep(WAIT_TIME);
 
             Bitmap bitmap = takeStatusBarScreenshot(mActivityRule.getActivity());
-            Stats s = evaluateDarkBarBitmap(bitmap, darkBackground, 0);
+            Stats s = evaluateDarkBarBitmap(bitmap, LIGHT_BG_COLOR, 0);
             assertStats(bitmap, s, false /* light */);
         });
     }
@@ -128,11 +127,10 @@
         assumeHasColoredStatusBar(mActivityRule);
 
         runInNotificationSession(() -> {
-            final int darkBackground = Color.rgb(128, 0, 0);
             final LightBarActivity activity = mActivityRule.getActivity();
             activity.runOnUiThread(() -> {
-                activity.getWindow().setStatusBarColor(darkBackground);
-                activity.getWindow().setNavigationBarColor(darkBackground);
+                activity.getWindow().setStatusBarColor(LIGHT_BG_COLOR);
+                activity.getWindow().setNavigationBarColor(LIGHT_BG_COLOR);
 
                 activity.setLightStatusBarAppearance(false);
                 activity.setLightNavigationBarAppearance(false);
@@ -145,7 +143,7 @@
             Thread.sleep(WAIT_TIME);
 
             Bitmap bitmap = takeStatusBarScreenshot(mActivityRule.getActivity());
-            Stats s = evaluateDarkBarBitmap(bitmap, darkBackground, 0);
+            Stats s = evaluateDarkBarBitmap(bitmap, LIGHT_BG_COLOR, 0);
             assertStats(bitmap, s, false /* light */);
         });
     }