Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove NEW_LIGHT_BAR_LOGIC flag" into udc-qpr-dev

Browse Source
This commit is contained in:
Jeff DeCew
2023-07-13 16:53:43 +00:00
committed by Android (Google) Code Review
parent 1ace020b79 29659a4ae4
commit 6daa509a40
6 changed files with 63 additions and 191 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/SystemUI/src/com/android/systemui/flags/Flags.kt
View File

@@ -61,10 +61,6 @@ object Flags {
// TODO(b/254512538): Tracking Bug // TODO(b/254512538): Tracking Bug
val INSTANT_VOICE_REPLY = unreleasedFlag(111, "instant_voice_reply") val INSTANT_VOICE_REPLY = unreleasedFlag(111, "instant_voice_reply")
// TODO(b/279735475): Tracking Bug
@JvmField
val NEW_LIGHT_BAR_LOGIC = releasedFlag(279735475, "new_light_bar_logic")
/** /**
* This flag is server-controlled and should stay as [unreleasedFlag] since we never want to * This flag is server-controlled and should stay as [unreleasedFlag] since we never want to
* enable it on release builds. * enable it on release builds.

169
packages/SystemUI/src/com/android/systemui/statusbar/phone/LightBarController.java
View File

@@ -38,8 +38,6 @@ import com.android.systemui.Dumpable;
import com.android.systemui.R; import com.android.systemui.R;
import com.android.systemui.dagger.SysUISingleton; import com.android.systemui.dagger.SysUISingleton;
import com.android.systemui.dump.DumpManager; import com.android.systemui.dump.DumpManager;
import com.android.systemui.flags.FeatureFlags;
import com.android.systemui.flags.Flags;
import com.android.systemui.navigationbar.NavigationModeController; import com.android.systemui.navigationbar.NavigationModeController;
import com.android.systemui.plugins.DarkIconDispatcher; import com.android.systemui.plugins.DarkIconDispatcher;
import com.android.systemui.settings.DisplayTracker; import com.android.systemui.settings.DisplayTracker;
@@ -65,7 +63,6 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
private final SysuiDarkIconDispatcher mStatusBarIconController; private final SysuiDarkIconDispatcher mStatusBarIconController;
private final BatteryController mBatteryController; private final BatteryController mBatteryController;
private final boolean mUseNewLightBarLogic;
private BiometricUnlockController mBiometricUnlockController; private BiometricUnlockController mBiometricUnlockController;
private LightBarTransitionsController mNavigationBarController; private LightBarTransitionsController mNavigationBarController;
@@ -123,10 +120,8 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
DarkIconDispatcher darkIconDispatcher, DarkIconDispatcher darkIconDispatcher,
BatteryController batteryController, BatteryController batteryController,
NavigationModeController navModeController, NavigationModeController navModeController,
FeatureFlags featureFlags,
DumpManager dumpManager, DumpManager dumpManager,
DisplayTracker displayTracker) { DisplayTracker displayTracker) {
mUseNewLightBarLogic = featureFlags.isEnabled(Flags.NEW_LIGHT_BAR_LOGIC);
mDarkIconColor = ctx.getColor(R.color.dark_mode_icon_color_single_tone); mDarkIconColor = ctx.getColor(R.color.dark_mode_icon_color_single_tone);
mLightIconColor = ctx.getColor(R.color.light_mode_icon_color_single_tone); mLightIconColor = ctx.getColor(R.color.light_mode_icon_color_single_tone);
mStatusBarIconController = (SysuiDarkIconDispatcher) darkIconDispatcher; mStatusBarIconController = (SysuiDarkIconDispatcher) darkIconDispatcher;
@@ -188,51 +183,31 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
final boolean last = mNavigationLight; final boolean last = mNavigationLight;
mHasLightNavigationBar = isLight(appearance, navigationBarMode, mHasLightNavigationBar = isLight(appearance, navigationBarMode,
APPEARANCE_LIGHT_NAVIGATION_BARS); APPEARANCE_LIGHT_NAVIGATION_BARS);
if (mUseNewLightBarLogic) { final boolean ignoreScrimForce = mDirectReplying && mNavbarColorManagedByIme;
final boolean ignoreScrimForce = mDirectReplying && mNavbarColorManagedByIme; final boolean darkForScrim = mForceDarkForScrim && !ignoreScrimForce;
final boolean darkForScrim = mForceDarkForScrim && !ignoreScrimForce; final boolean lightForScrim = mForceLightForScrim && !ignoreScrimForce;
final boolean lightForScrim = mForceLightForScrim && !ignoreScrimForce; final boolean darkForQs = (mQsCustomizing || mQsExpanded) && !mBouncerVisible;
final boolean darkForQs = (mQsCustomizing || mQsExpanded) && !mBouncerVisible; final boolean darkForTop = darkForQs || mGlobalActionsVisible;
final boolean darkForTop = darkForQs || mGlobalActionsVisible; mNavigationLight =
mNavigationLight = ((mHasLightNavigationBar && !darkForScrim) || lightForScrim) && !darkForTop;
((mHasLightNavigationBar && !darkForScrim) || lightForScrim) && !darkForTop; if (DEBUG_NAVBAR) {
if (DEBUG_NAVBAR) { mLastNavigationBarAppearanceChangedLog = getLogStringBuilder()
mLastNavigationBarAppearanceChangedLog = getLogStringBuilder() .append("onNavigationBarAppearanceChanged()")
.append("onNavigationBarAppearanceChanged()") .append(" appearance=").append(appearance)
.append(" appearance=").append(appearance) .append(" nbModeChanged=").append(nbModeChanged)
.append(" nbModeChanged=").append(nbModeChanged) .append(" navigationBarMode=").append(navigationBarMode)
.append(" navigationBarMode=").append(navigationBarMode) .append(" navbarColorManagedByIme=").append(navbarColorManagedByIme)
.append(" navbarColorManagedByIme=").append(navbarColorManagedByIme) .append(" mHasLightNavigationBar=").append(mHasLightNavigationBar)
.append(" mHasLightNavigationBar=").append(mHasLightNavigationBar) .append(" ignoreScrimForce=").append(ignoreScrimForce)
.append(" ignoreScrimForce=").append(ignoreScrimForce) .append(" darkForScrim=").append(darkForScrim)
.append(" darkForScrim=").append(darkForScrim) .append(" lightForScrim=").append(lightForScrim)
.append(" lightForScrim=").append(lightForScrim) .append(" darkForQs=").append(darkForQs)
.append(" darkForQs=").append(darkForQs) .append(" darkForTop=").append(darkForTop)
.append(" darkForTop=").append(darkForTop) .append(" mNavigationLight=").append(mNavigationLight)
.append(" mNavigationLight=").append(mNavigationLight) .append(" last=").append(last)
.append(" last=").append(last) .append(" timestamp=").append(System.currentTimeMillis())
.append(" timestamp=").append(System.currentTimeMillis()) .toString();
.toString(); if (DEBUG_LOGS) Log.d(TAG, mLastNavigationBarAppearanceChangedLog);
if (DEBUG_LOGS) Log.d(TAG, mLastNavigationBarAppearanceChangedLog);
}
} else {
mNavigationLight = mHasLightNavigationBar
&& (mDirectReplying && mNavbarColorManagedByIme || !mForceDarkForScrim)
&& !mQsCustomizing;
if (DEBUG_NAVBAR) {
mLastNavigationBarAppearanceChangedLog = getLogStringBuilder()
.append("onNavigationBarAppearanceChanged()")
.append(" appearance=").append(appearance)
.append(" nbModeChanged=").append(nbModeChanged)
.append(" navigationBarMode=").append(navigationBarMode)
.append(" navbarColorManagedByIme=").append(navbarColorManagedByIme)
.append(" mHasLightNavigationBar=").append(mHasLightNavigationBar)
.append(" mNavigationLight=").append(mNavigationLight)
.append(" last=").append(last)
.append(" timestamp=").append(System.currentTimeMillis())
.toString();
if (DEBUG_LOGS) Log.d(TAG, mLastNavigationBarAppearanceChangedLog);
}
} }
if (mNavigationLight != last) { if (mNavigationLight != last) {
updateNavigation(); updateNavigation();
@@ -311,66 +286,39 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
public void setScrimState(ScrimState scrimState, float scrimBehindAlpha, public void setScrimState(ScrimState scrimState, float scrimBehindAlpha,
GradientColors scrimInFrontColor) { GradientColors scrimInFrontColor) {
if (mUseNewLightBarLogic) { boolean bouncerVisibleLast = mBouncerVisible;
boolean bouncerVisibleLast = mBouncerVisible; boolean forceDarkForScrimLast = mForceDarkForScrim;
boolean forceDarkForScrimLast = mForceDarkForScrim; boolean forceLightForScrimLast = mForceLightForScrim;
boolean forceLightForScrimLast = mForceLightForScrim; mBouncerVisible =
mBouncerVisible = scrimState == ScrimState.BOUNCER || scrimState == ScrimState.BOUNCER_SCRIMMED;
scrimState == ScrimState.BOUNCER || scrimState == ScrimState.BOUNCER_SCRIMMED; final boolean forceForScrim = mBouncerVisible
final boolean forceForScrim = mBouncerVisible || scrimBehindAlpha >= NAV_BAR_INVERSION_SCRIM_ALPHA_THRESHOLD;
|| scrimBehindAlpha >= NAV_BAR_INVERSION_SCRIM_ALPHA_THRESHOLD; final boolean scrimColorIsLight = scrimInFrontColor.supportsDarkText();
final boolean scrimColorIsLight = scrimInFrontColor.supportsDarkText();
mForceDarkForScrim = forceForScrim && !scrimColorIsLight; mForceDarkForScrim = forceForScrim && !scrimColorIsLight;
mForceLightForScrim = forceForScrim && scrimColorIsLight; mForceLightForScrim = forceForScrim && scrimColorIsLight;
if (mBouncerVisible != bouncerVisibleLast) { if (mBouncerVisible != bouncerVisibleLast) {
reevaluate(); reevaluate();
} else if (mHasLightNavigationBar) { } else if (mHasLightNavigationBar) {
if (mForceDarkForScrim != forceDarkForScrimLast) reevaluate(); if (mForceDarkForScrim != forceDarkForScrimLast) reevaluate();
} else {
if (mForceLightForScrim != forceLightForScrimLast) reevaluate();
}
if (DEBUG_NAVBAR) {
mLastSetScrimStateLog = getLogStringBuilder()
.append("setScrimState()")
.append(" scrimState=").append(scrimState)
.append(" scrimBehindAlpha=").append(scrimBehindAlpha)
.append(" scrimInFrontColor=").append(scrimInFrontColor)
.append(" forceForScrim=").append(forceForScrim)
.append(" scrimColorIsLight=").append(scrimColorIsLight)
.append(" mHasLightNavigationBar=").append(mHasLightNavigationBar)
.append(" mBouncerVisible=").append(mBouncerVisible)
.append(" mForceDarkForScrim=").append(mForceDarkForScrim)
.append(" mForceLightForScrim=").append(mForceLightForScrim)
.append(" timestamp=").append(System.currentTimeMillis())
.toString();
if (DEBUG_LOGS) Log.d(TAG, mLastSetScrimStateLog);
}
} else { } else {
boolean forceDarkForScrimLast = mForceDarkForScrim; if (mForceLightForScrim != forceLightForScrimLast) reevaluate();
// For BOUNCER/BOUNCER_SCRIMMED cases, we assume that alpha is always below threshold. }
// This enables IMEs to control the navigation bar color. if (DEBUG_NAVBAR) {
// For other cases, scrim should be able to veto the light navigation bar. mLastSetScrimStateLog = getLogStringBuilder()
// NOTE: this was also wrong for S and has been removed in the new logic. .append("setScrimState()")
mForceDarkForScrim = scrimState != ScrimState.BOUNCER .append(" scrimState=").append(scrimState)
&& scrimState != ScrimState.BOUNCER_SCRIMMED .append(" scrimBehindAlpha=").append(scrimBehindAlpha)
&& scrimBehindAlpha >= NAV_BAR_INVERSION_SCRIM_ALPHA_THRESHOLD .append(" scrimInFrontColor=").append(scrimInFrontColor)
&& !scrimInFrontColor.supportsDarkText(); .append(" forceForScrim=").append(forceForScrim)
if (mHasLightNavigationBar && (mForceDarkForScrim != forceDarkForScrimLast)) { .append(" scrimColorIsLight=").append(scrimColorIsLight)
reevaluate(); .append(" mHasLightNavigationBar=").append(mHasLightNavigationBar)
} .append(" mBouncerVisible=").append(mBouncerVisible)
if (DEBUG_NAVBAR) { .append(" mForceDarkForScrim=").append(mForceDarkForScrim)
mLastSetScrimStateLog = getLogStringBuilder() .append(" mForceLightForScrim=").append(mForceLightForScrim)
.append("setScrimState()") .append(" timestamp=").append(System.currentTimeMillis())
.append(" scrimState=").append(scrimState) .toString();
.append(" scrimBehindAlpha=").append(scrimBehindAlpha) if (DEBUG_LOGS) Log.d(TAG, mLastSetScrimStateLog);
.append(" scrimInFrontColor=").append(scrimInFrontColor)
.append(" mHasLightNavigationBar=").append(mHasLightNavigationBar)
.append(" mForceDarkForScrim=").append(mForceDarkForScrim)
.append(" timestamp=").append(System.currentTimeMillis())
.toString();
if (DEBUG_LOGS) Log.d(TAG, mLastSetScrimStateLog);
}
} }
} }
@@ -498,7 +446,6 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
private final DarkIconDispatcher mDarkIconDispatcher; private final DarkIconDispatcher mDarkIconDispatcher;
private final BatteryController mBatteryController; private final BatteryController mBatteryController;
private final NavigationModeController mNavModeController; private final NavigationModeController mNavModeController;
private final FeatureFlags mFeatureFlags;
private final DumpManager mDumpManager; private final DumpManager mDumpManager;
private final DisplayTracker mDisplayTracker; private final DisplayTracker mDisplayTracker;
@@ -507,14 +454,12 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
DarkIconDispatcher darkIconDispatcher, DarkIconDispatcher darkIconDispatcher,
BatteryController batteryController, BatteryController batteryController,
NavigationModeController navModeController, NavigationModeController navModeController,
FeatureFlags featureFlags,
DumpManager dumpManager, DumpManager dumpManager,
DisplayTracker displayTracker) { DisplayTracker displayTracker) {
mDarkIconDispatcher = darkIconDispatcher; mDarkIconDispatcher = darkIconDispatcher;
mBatteryController = batteryController; mBatteryController = batteryController;
mNavModeController = navModeController; mNavModeController = navModeController;
mFeatureFlags = featureFlags;
mDumpManager = dumpManager; mDumpManager = dumpManager;
mDisplayTracker = displayTracker; mDisplayTracker = displayTracker;
} }
@@ -522,7 +467,7 @@ public class LightBarController implements BatteryController.BatteryStateChangeC
/** Create an {@link LightBarController} */ /** Create an {@link LightBarController} */
public LightBarController create(Context context) { public LightBarController create(Context context) {
return new LightBarController(context, mDarkIconDispatcher, mBatteryController, return new LightBarController(context, mDarkIconDispatcher, mBatteryController,
mNavModeController, mFeatureFlags, mDumpManager, mDisplayTracker); mNavModeController, mDumpManager, mDisplayTracker);
} }
} }
} }

28
packages/SystemUI/src/com/android/systemui/statusbar/phone/ScrimController.java
View File

@@ -56,8 +56,6 @@ import com.android.systemui.bouncer.shared.constants.KeyguardBouncerConstants;
import com.android.systemui.dagger.SysUISingleton; import com.android.systemui.dagger.SysUISingleton;
import com.android.systemui.dagger.qualifiers.Main; import com.android.systemui.dagger.qualifiers.Main;
import com.android.systemui.dock.DockManager; import com.android.systemui.dock.DockManager;
import com.android.systemui.flags.FeatureFlags;
import com.android.systemui.flags.Flags;
import com.android.systemui.keyguard.KeyguardUnlockAnimationController; import com.android.systemui.keyguard.KeyguardUnlockAnimationController;
import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor; import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor;
import com.android.systemui.keyguard.shared.model.ScrimAlpha; import com.android.systemui.keyguard.shared.model.ScrimAlpha;
@@ -249,8 +247,6 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
private int mScrimsVisibility; private int mScrimsVisibility;
private final TriConsumer<ScrimState, Float, GradientColors> mScrimStateListener; private final TriConsumer<ScrimState, Float, GradientColors> mScrimStateListener;
private final LargeScreenShadeInterpolator mLargeScreenShadeInterpolator; private final LargeScreenShadeInterpolator mLargeScreenShadeInterpolator;
private final FeatureFlags mFeatureFlags;
private final boolean mUseNewLightBarLogic;
private Consumer<Integer> mScrimVisibleListener; private Consumer<Integer> mScrimVisibleListener;
private boolean mBlankScreen; private boolean mBlankScreen;
private boolean mScreenBlankingCallbackCalled; private boolean mScreenBlankingCallbackCalled;
@@ -303,12 +299,9 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
PrimaryBouncerToGoneTransitionViewModel primaryBouncerToGoneTransitionViewModel, PrimaryBouncerToGoneTransitionViewModel primaryBouncerToGoneTransitionViewModel,
KeyguardTransitionInteractor keyguardTransitionInteractor, KeyguardTransitionInteractor keyguardTransitionInteractor,
@Main CoroutineDispatcher mainDispatcher, @Main CoroutineDispatcher mainDispatcher,
LargeScreenShadeInterpolator largeScreenShadeInterpolator, LargeScreenShadeInterpolator largeScreenShadeInterpolator) {
FeatureFlags featureFlags) {
mScrimStateListener = lightBarController::setScrimState; mScrimStateListener = lightBarController::setScrimState;
mLargeScreenShadeInterpolator = largeScreenShadeInterpolator; mLargeScreenShadeInterpolator = largeScreenShadeInterpolator;
mFeatureFlags = featureFlags;
mUseNewLightBarLogic = featureFlags.isEnabled(Flags.NEW_LIGHT_BAR_LOGIC);
mDefaultScrimAlpha = BUSY_SCRIM_ALPHA; mDefaultScrimAlpha = BUSY_SCRIM_ALPHA;
mKeyguardStateController = keyguardStateController; mKeyguardStateController = keyguardStateController;
@@ -1153,13 +1146,7 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
if (mClipsQsScrim && mQsBottomVisible) { if (mClipsQsScrim && mQsBottomVisible) {
alpha = mNotificationsAlpha; alpha = mNotificationsAlpha;
} }
if (mUseNewLightBarLogic) { mScrimStateListener.accept(mState, alpha, mColors);
mScrimStateListener.accept(mState, alpha, mColors);
} else {
// NOTE: This wasn't wrong, but it implied that each scrim might have different colors,
// when in fact they all share the same GradientColors instance, which we own.
mScrimStateListener.accept(mState, alpha, mScrimInFront.getColors());
}
} }
private void dispatchScrimsVisible() { private void dispatchScrimsVisible() {
@@ -1483,15 +1470,8 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
int accent = Utils.getColorAccent(mScrimBehind.getContext()).getDefaultColor(); int accent = Utils.getColorAccent(mScrimBehind.getContext()).getDefaultColor();
mColors.setMainColor(background); mColors.setMainColor(background);
mColors.setSecondaryColor(accent); mColors.setSecondaryColor(accent);
if (mUseNewLightBarLogic) { final boolean isBackgroundLight = !ContrastColorUtil.isColorDark(background);
final boolean isBackgroundLight = !ContrastColorUtil.isColorDark(background); mColors.setSupportsDarkText(isBackgroundLight);
mColors.setSupportsDarkText(isBackgroundLight);
} else {
// NOTE: This was totally backward, but LightBarController was flipping it back.
// There may be other consumers of this which would struggle though
mColors.setSupportsDarkText(
ColorUtils.calculateContrast(mColors.getMainColor(), Color.WHITE) > 4.5);
}
int surface = Utils.getColorAttr(mScrimBehind.getContext(), int surface = Utils.getColorAttr(mScrimBehind.getContext(),
com.android.internal.R.attr.materialColorSurface).getDefaultColor(); com.android.internal.R.attr.materialColorSurface).getDefaultColor();

17
packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/LightBarControllerTest.java
View File

@@ -22,7 +22,6 @@ import static com.android.systemui.statusbar.phone.BarTransitions.MODE_TRANSPARE
import static junit.framework.Assert.assertTrue; import static junit.framework.Assert.assertTrue;
import static org.junit.Assume.assumeTrue;
import static org.mockito.ArgumentMatchers.anyBoolean; import static org.mockito.ArgumentMatchers.anyBoolean;
import static org.mockito.ArgumentMatchers.anyInt; import static org.mockito.ArgumentMatchers.anyInt;
import static org.mockito.ArgumentMatchers.eq; import static org.mockito.ArgumentMatchers.eq;
@@ -45,8 +44,6 @@ import com.android.internal.util.ContrastColorUtil;
import com.android.internal.view.AppearanceRegion; import com.android.internal.view.AppearanceRegion;
import com.android.systemui.SysuiTestCase; import com.android.systemui.SysuiTestCase;
import com.android.systemui.dump.DumpManager; import com.android.systemui.dump.DumpManager;
import com.android.systemui.flags.FakeFeatureFlags;
import com.android.systemui.flags.Flags;
import com.android.systemui.navigationbar.NavigationModeController; import com.android.systemui.navigationbar.NavigationModeController;
import com.android.systemui.settings.FakeDisplayTracker; import com.android.systemui.settings.FakeDisplayTracker;
import com.android.systemui.statusbar.policy.BatteryController; import com.android.systemui.statusbar.policy.BatteryController;
@@ -65,20 +62,13 @@ public class LightBarControllerTest extends SysuiTestCase {
private static final GradientColors COLORS_LIGHT = makeColors(Color.WHITE); private static final GradientColors COLORS_LIGHT = makeColors(Color.WHITE);
private static final GradientColors COLORS_DARK = makeColors(Color.BLACK); private static final GradientColors COLORS_DARK = makeColors(Color.BLACK);
private final FakeFeatureFlags mFeatureFlags = new FakeFeatureFlags();
private LightBarTransitionsController mLightBarTransitionsController; private LightBarTransitionsController mLightBarTransitionsController;
private LightBarTransitionsController mNavBarController; private LightBarTransitionsController mNavBarController;
private SysuiDarkIconDispatcher mStatusBarIconController; private SysuiDarkIconDispatcher mStatusBarIconController;
private LightBarController mLightBarController; private LightBarController mLightBarController;
/** Allow testing with NEW_LIGHT_BAR_LOGIC flag in different states */
protected boolean testNewLightBarLogic() {
return false;
}
@Before @Before
public void setup() { public void setup() {
mFeatureFlags.set(Flags.NEW_LIGHT_BAR_LOGIC, testNewLightBarLogic());
mStatusBarIconController = mock(SysuiDarkIconDispatcher.class); mStatusBarIconController = mock(SysuiDarkIconDispatcher.class);
mNavBarController = mock(LightBarTransitionsController.class); mNavBarController = mock(LightBarTransitionsController.class);
when(mNavBarController.supportsIconTintForNavMode(anyInt())).thenReturn(true); when(mNavBarController.supportsIconTintForNavMode(anyInt())).thenReturn(true);
@@ -90,7 +80,6 @@ public class LightBarControllerTest extends SysuiTestCase {
mStatusBarIconController, mStatusBarIconController,
mock(BatteryController.class), mock(BatteryController.class),
mock(NavigationModeController.class), mock(NavigationModeController.class),
mFeatureFlags,
mock(DumpManager.class), mock(DumpManager.class),
new FakeDisplayTracker(mContext)); new FakeDisplayTracker(mContext));
} }
@@ -211,8 +200,6 @@ public class LightBarControllerTest extends SysuiTestCase {
@Test @Test
public void validateNavBarChangesUpdateIcons() { public void validateNavBarChangesUpdateIcons() {
assumeTrue(testNewLightBarLogic()); // Only run in the new suite
// On the launcher in dark mode buttons are light // On the launcher in dark mode buttons are light
mLightBarController.setScrimState(ScrimState.UNLOCKED, 0f, COLORS_DARK); mLightBarController.setScrimState(ScrimState.UNLOCKED, 0f, COLORS_DARK);
mLightBarController.onNavigationBarAppearanceChanged( mLightBarController.onNavigationBarAppearanceChanged(
@@ -251,8 +238,6 @@ public class LightBarControllerTest extends SysuiTestCase {
@Test @Test
public void navBarHasDarkIconsInLockedShade_lightMode() { public void navBarHasDarkIconsInLockedShade_lightMode() {
assumeTrue(testNewLightBarLogic()); // Only run in the new suite
// On the locked shade QS in light mode buttons are light // On the locked shade QS in light mode buttons are light
mLightBarController.setScrimState(ScrimState.SHADE_LOCKED, 1f, COLORS_LIGHT); mLightBarController.setScrimState(ScrimState.SHADE_LOCKED, 1f, COLORS_LIGHT);
mLightBarController.onNavigationBarAppearanceChanged( mLightBarController.onNavigationBarAppearanceChanged(
@@ -287,8 +272,6 @@ public class LightBarControllerTest extends SysuiTestCase {
@Test @Test
public void navBarHasLightIconsInLockedShade_darkMode() { public void navBarHasLightIconsInLockedShade_darkMode() {
assumeTrue(testNewLightBarLogic()); // Only run in the new suite
// On the locked shade QS in light mode buttons are light // On the locked shade QS in light mode buttons are light
mLightBarController.setScrimState(ScrimState.SHADE_LOCKED, 1f, COLORS_DARK); mLightBarController.setScrimState(ScrimState.SHADE_LOCKED, 1f, COLORS_DARK);
mLightBarController.onNavigationBarAppearanceChanged( mLightBarController.onNavigationBarAppearanceChanged(

28
packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/LightBarControllerWithNewLogicTest.kt
View File

@@ -1,28 +0,0 @@
/*
* Copyright (C) 2023 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.android.systemui.statusbar.phone
import androidx.test.filters.SmallTest
import com.android.systemui.flags.Flags.NEW_LIGHT_BAR_LOGIC
/**
* This file only needs to live as long as [NEW_LIGHT_BAR_LOGIC] does. When we delete that flag, we
* can roll this back into the old test.
*/
@SmallTest
class LightBarControllerWithNewLogicTest : LightBarControllerTest() {
override fun testNewLightBarLogic(): Boolean = true
}

8
packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/ScrimControllerTest.java
View File

@@ -64,7 +64,6 @@ import com.android.systemui.SysuiTestCase;
import com.android.systemui.animation.ShadeInterpolation; import com.android.systemui.animation.ShadeInterpolation;
import com.android.systemui.bouncer.shared.constants.KeyguardBouncerConstants; import com.android.systemui.bouncer.shared.constants.KeyguardBouncerConstants;
import com.android.systemui.dock.DockManager; import com.android.systemui.dock.DockManager;
import com.android.systemui.flags.FeatureFlags;
import com.android.systemui.keyguard.KeyguardUnlockAnimationController; import com.android.systemui.keyguard.KeyguardUnlockAnimationController;
import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor; import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor;
import com.android.systemui.keyguard.shared.model.KeyguardState; import com.android.systemui.keyguard.shared.model.KeyguardState;
@@ -142,7 +141,6 @@ public class ScrimControllerTest extends SysuiTestCase {
// TODO(b/204991468): Use a real PanelExpansionStateManager object once this bug is fixed. (The // TODO(b/204991468): Use a real PanelExpansionStateManager object once this bug is fixed. (The
// event-dispatch-on-registration pattern caused some of these unit tests to fail.) // event-dispatch-on-registration pattern caused some of these unit tests to fail.)
@Mock private StatusBarKeyguardViewManager mStatusBarKeyguardViewManager; @Mock private StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;
@Mock private FeatureFlags mFeatureFlags;
private static class AnimatorListener implements Animator.AnimatorListener { private static class AnimatorListener implements Animator.AnimatorListener {
private int mNumStarts; private int mNumStarts;
@@ -280,8 +278,7 @@ public class ScrimControllerTest extends SysuiTestCase {
mPrimaryBouncerToGoneTransitionViewModel, mPrimaryBouncerToGoneTransitionViewModel,
mKeyguardTransitionInteractor, mKeyguardTransitionInteractor,
mMainDispatcher, mMainDispatcher,
mLinearLargeScreenShadeInterpolator, mLinearLargeScreenShadeInterpolator);
mFeatureFlags);
mScrimController.setScrimVisibleListener(visible -> mScrimVisibility = visible); mScrimController.setScrimVisibleListener(visible -> mScrimVisibility = visible);
mScrimController.attachViews(mScrimBehind, mNotificationsScrim, mScrimInFront); mScrimController.attachViews(mScrimBehind, mNotificationsScrim, mScrimInFront);
mScrimController.setAnimatorListener(mAnimatorListener); mScrimController.setAnimatorListener(mAnimatorListener);
@@ -974,8 +971,7 @@ public class ScrimControllerTest extends SysuiTestCase {
mPrimaryBouncerToGoneTransitionViewModel, mPrimaryBouncerToGoneTransitionViewModel,
mKeyguardTransitionInteractor, mKeyguardTransitionInteractor,
mMainDispatcher, mMainDispatcher,
mLinearLargeScreenShadeInterpolator, mLinearLargeScreenShadeInterpolator);
mFeatureFlags);
mScrimController.setScrimVisibleListener(visible -> mScrimVisibility = visible); mScrimController.setScrimVisibleListener(visible -> mScrimVisibility = visible);
mScrimController.attachViews(mScrimBehind, mNotificationsScrim, mScrimInFront); mScrimController.attachViews(mScrimBehind, mNotificationsScrim, mScrimInFront);
mScrimController.setAnimatorListener(mAnimatorListener); mScrimController.setAnimatorListener(mAnimatorListener);