Merge "On keyguard unoccluding, set notif scrim alpha to KEYGUARD_SCRIM_ALPHA" into sc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
0f2a769d79
@@ -711,7 +711,8 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
|
|||||||
}
|
}
|
||||||
if (mUnOcclusionAnimationRunning && mState == ScrimState.KEYGUARD) {
|
if (mUnOcclusionAnimationRunning && mState == ScrimState.KEYGUARD) {
|
||||||
// We're unoccluding the keyguard and don't want to have a bright flash.
|
// We're unoccluding the keyguard and don't want to have a bright flash.
|
||||||
mNotificationsAlpha = 0f;
|
mNotificationsAlpha = KEYGUARD_SCRIM_ALPHA;
|
||||||
|
mNotificationsTint = ScrimState.KEYGUARD.getNotifTint();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (isNaN(mBehindAlpha) || isNaN(mInFrontAlpha) || isNaN(mNotificationsAlpha)) {
|
if (isNaN(mBehindAlpha) || isNaN(mInFrontAlpha) || isNaN(mNotificationsAlpha)) {
|
||||||
|
|||||||
@@ -16,6 +16,7 @@
|
|||||||
|
|
||||||
package com.android.systemui.statusbar.phone;
|
package com.android.systemui.statusbar.phone;
|
||||||
|
|
||||||
|
import static com.android.systemui.statusbar.phone.ScrimController.KEYGUARD_SCRIM_ALPHA;
|
||||||
import static com.android.systemui.statusbar.phone.ScrimController.OPAQUE;
|
import static com.android.systemui.statusbar.phone.ScrimController.OPAQUE;
|
||||||
import static com.android.systemui.statusbar.phone.ScrimController.SEMI_TRANSPARENT;
|
import static com.android.systemui.statusbar.phone.ScrimController.SEMI_TRANSPARENT;
|
||||||
import static com.android.systemui.statusbar.phone.ScrimController.TRANSPARENT;
|
import static com.android.systemui.statusbar.phone.ScrimController.TRANSPARENT;
|
||||||
@@ -1186,8 +1187,10 @@ public class ScrimControllerTest extends SysuiTestCase {
|
|||||||
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
||||||
mScrimController.setUnocclusionAnimationRunning(true);
|
mScrimController.setUnocclusionAnimationRunning(true);
|
||||||
|
|
||||||
assertAlphaAfterExpansion(mNotificationsScrim, /* alpha */ 0.0f, /* expansion */ 0.0f);
|
assertAlphaAfterExpansion(mNotificationsScrim, /* alpha */ KEYGUARD_SCRIM_ALPHA,
|
||||||
assertAlphaAfterExpansion(mNotificationsScrim, /* alpha */ 0.0f, /* expansion */ 1.0f);
|
/* expansion */ 0.0f);
|
||||||
|
assertAlphaAfterExpansion(mNotificationsScrim, /* alpha */ KEYGUARD_SCRIM_ALPHA,
|
||||||
|
/* expansion */ 1.0f);
|
||||||
|
|
||||||
// Verify normal behavior after
|
// Verify normal behavior after
|
||||||
mScrimController.setUnocclusionAnimationRunning(false);
|
mScrimController.setUnocclusionAnimationRunning(false);
|
||||||
|
|||||||
Reference in New Issue
Block a user