Merge "Bouncer should dismiss if being tracked when face unlocked" into tm-d1-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
e12369e3cf
@@ -2692,7 +2692,7 @@ public class KeyguardViewMediator extends CoreStartable implements Dumpable,
|
||||
/** Hides the surface behind the keyguard by re-showing the keyguard/activity lock screen. */
|
||||
public void hideSurfaceBehindKeyguard() {
|
||||
mSurfaceBehindRemoteAnimationRequested = false;
|
||||
|
||||
mKeyguardStateController.notifyKeyguardGoingAway(false);
|
||||
if (mShowing) {
|
||||
setShowingLocked(true, true);
|
||||
}
|
||||
|
||||
@@ -397,10 +397,6 @@ public class KeyguardBouncer {
|
||||
return mShowingSoon || mExpansion != EXPANSION_HIDDEN && mExpansion != EXPANSION_VISIBLE;
|
||||
}
|
||||
|
||||
public boolean getShowingSoon() {
|
||||
return mShowingSoon;
|
||||
}
|
||||
|
||||
/**
|
||||
* @return {@code true} when bouncer's pre-hide animation already started but isn't completely
|
||||
* hidden yet, {@code false} otherwise.
|
||||
|
||||
@@ -963,7 +963,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
|
||||
|
||||
@Override
|
||||
public boolean bouncerIsOrWillBeShowing() {
|
||||
return isBouncerShowing() || mBouncer.getShowingSoon();
|
||||
return isBouncerShowing() || mBouncer.inTransit();
|
||||
}
|
||||
|
||||
public boolean isFullscreenBouncer() {
|
||||
|
||||
@@ -380,6 +380,16 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase {
|
||||
verify(mAlternateAuthInterceptor).hideAlternateAuthBouncer();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testBouncerIsOrWillBeShowing_whenBouncerIsInTransit() {
|
||||
when(mBouncer.isShowing()).thenReturn(false);
|
||||
when(mBouncer.inTransit()).thenReturn(true);
|
||||
|
||||
assertTrue(
|
||||
"Is or will be showing should be true when bouncer is in transit",
|
||||
mStatusBarKeyguardViewManager.bouncerIsOrWillBeShowing());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testShowAltAuth_unlockingWithBiometricNotAllowed() {
|
||||
// GIVEN alt auth exists, unlocking with biometric isn't allowed
|
||||
|
||||
Reference in New Issue
Block a user