diff options
author | Chandru S <chandruis@google.com> | 2023-02-07 15:26:15 -0800 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2023-02-22 07:19:21 +0000 |
commit | 68d30e00bb32753e07c96ef46e470ef5a9375f59 (patch) | |
tree | eb49c15b08e376d729c8c94635f163ed45cbd785 | |
parent | 9d0bf9c6d5a45f98256122dbdc52b04be77c4197 (diff) | |
download | base-68d30e00bb32753e07c96ef46e470ef5a9375f59.tar.gz |
Use alternateBouncerShowing instead of mOccludingAppRequestingFace to trigger face auth while secure camera is launched.
- mOccludingAppRequestingFace is set to true by `UdfpsKeyguardViewController#showUdfpsBouncer`
- This doesn't happen for devices with non-UDFPS fingerpint sensors.
- Using alternateBouncerShowing would work better here.
Other changes:
- Renames udfpsBouncerShowing to alternateBouncerShowing
Fixes: 267132953
Test: atest KeyguardUpdateMonitorTest
Test: manually,
1. Enroll fingerprint (non-udfps) & face
2. Suspend phone and go to Aod/Lockscreen
3. Double press power button to open camera over lockscreen
4. Open the gallery from the camera app.
5. Alternate FPS bouncer should be shown and face auth should also run
Change-Id: I05d4256169a6fe491ed3c3be1cd6690341ad931f
(cherry picked from commit a9443a091b05bee936925d15a219bad9f211a1d5)
Merged-In: I05d4256169a6fe491ed3c3be1cd6690341ad931f
(cherry picked from commit 7a86edb64745acaee75714eba7b3b11210e315a3)
Merged-In: I05d4256169a6fe491ed3c3be1cd6690341ad931f
4 files changed, 30 insertions, 26 deletions
diff --git a/packages/SystemUI/src/com/android/keyguard/KeyguardFaceListenModel.kt b/packages/SystemUI/src/com/android/keyguard/KeyguardFaceListenModel.kt index 1a06b5f1c767..fe8b8c944d13 100644 --- a/packages/SystemUI/src/com/android/keyguard/KeyguardFaceListenModel.kt +++ b/packages/SystemUI/src/com/android/keyguard/KeyguardFaceListenModel.kt @@ -27,6 +27,7 @@ data class KeyguardFaceListenModel( override var userId: Int = 0, override var listening: Boolean = false, // keep sorted + var alternateBouncerShowing: Boolean = false, var authInterruptActive: Boolean = false, var biometricSettingEnabledForUser: Boolean = false, var bouncerFullyShown: Boolean = false, @@ -44,7 +45,6 @@ data class KeyguardFaceListenModel( var secureCameraLaunched: Boolean = false, var supportsDetect: Boolean = false, var switchingUser: Boolean = false, - var udfpsBouncerShowing: Boolean = false, var udfpsFingerDown: Boolean = false, var userNotTrustedOrDetectionIsNeeded: Boolean = false, ) : KeyguardListenModel() { @@ -73,7 +73,7 @@ data class KeyguardFaceListenModel( secureCameraLaunched.toString(), supportsDetect.toString(), switchingUser.toString(), - udfpsBouncerShowing.toString(), + alternateBouncerShowing.toString(), udfpsFingerDown.toString(), userNotTrustedOrDetectionIsNeeded.toString(), ) @@ -95,6 +95,7 @@ data class KeyguardFaceListenModel( userId = model.userId listening = model.listening // keep sorted + alternateBouncerShowing = model.alternateBouncerShowing biometricSettingEnabledForUser = model.biometricSettingEnabledForUser bouncerFullyShown = model.bouncerFullyShown faceAndFpNotAuthenticated = model.faceAndFpNotAuthenticated @@ -111,7 +112,6 @@ data class KeyguardFaceListenModel( secureCameraLaunched = model.secureCameraLaunched supportsDetect = model.supportsDetect switchingUser = model.switchingUser - udfpsBouncerShowing = model.udfpsBouncerShowing switchingUser = model.switchingUser udfpsFingerDown = model.udfpsFingerDown userNotTrustedOrDetectionIsNeeded = model.userNotTrustedOrDetectionIsNeeded diff --git a/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java b/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java index e23042f107eb..935c7c2d85a1 100644 --- a/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java +++ b/packages/SystemUI/src/com/android/keyguard/KeyguardUpdateMonitor.java @@ -310,7 +310,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab private boolean mGoingToSleep; private boolean mPrimaryBouncerFullyShown; private boolean mPrimaryBouncerIsOrWillBeShowing; - private boolean mUdfpsBouncerShowing; + private boolean mAlternateBouncerShowing; private boolean mAuthInterruptActive; private boolean mNeedsSlowUnlockTransition; private boolean mAssistantVisible; @@ -535,7 +535,8 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab * It's assumed that the trust was granted for the current user. */ private boolean shouldDismissKeyguardOnTrustGrantedWithCurrentUser(TrustGrantFlags flags) { - final boolean isBouncerShowing = mPrimaryBouncerIsOrWillBeShowing || mUdfpsBouncerShowing; + final boolean isBouncerShowing = + mPrimaryBouncerIsOrWillBeShowing || mAlternateBouncerShowing; return (flags.isInitiatedByUser() || flags.dismissKeyguardRequested()) && (mDeviceInteractive || flags.temporaryAndRenewable()) && (isBouncerShowing || flags.dismissKeyguardRequested()); @@ -1729,7 +1730,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab public void onAuthenticationFailed() { String reason = mKeyguardBypassController.canBypass() ? "bypass" - : mUdfpsBouncerShowing ? "udfpsBouncer" + : mAlternateBouncerShowing ? "alternateBouncer" : mPrimaryBouncerFullyShown ? "bouncer" : "udfpsFpDown"; requestActiveUnlock( @@ -2585,7 +2586,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab requestActiveUnlock( requestOrigin, extraReason, canFaceBypass - || mUdfpsBouncerShowing + || mAlternateBouncerShowing || mPrimaryBouncerFullyShown || mAuthController.isUdfpsFingerDown()); } @@ -2603,16 +2604,16 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab } /** - * Whether the UDFPS bouncer is showing. + * Whether the alternate bouncer is showing. */ - public void setUdfpsBouncerShowing(boolean showing) { - mUdfpsBouncerShowing = showing; - if (mUdfpsBouncerShowing) { + public void setAlternateBouncerShowing(boolean showing) { + mAlternateBouncerShowing = showing; + if (mAlternateBouncerShowing) { updateFaceListeningState(BIOMETRIC_ACTION_START, FACE_AUTH_TRIGGERED_ALTERNATE_BIOMETRIC_BOUNCER_SHOWN); requestActiveUnlock( ActiveUnlockConfig.ACTIVE_UNLOCK_REQUEST_ORIGIN.UNLOCK_INTENT, - "udfpsBouncer"); + "alternateBouncer"); } updateFingerprintListeningState(BIOMETRIC_ACTION_UPDATE); } @@ -2620,7 +2621,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab private boolean shouldTriggerActiveUnlock() { // Triggers: final boolean triggerActiveUnlockForAssistant = shouldTriggerActiveUnlockForAssistant(); - final boolean awakeKeyguard = mPrimaryBouncerFullyShown || mUdfpsBouncerShowing + final boolean awakeKeyguard = mPrimaryBouncerFullyShown || mAlternateBouncerShowing || (isKeyguardVisible() && !mGoingToSleep && mStatusBarState != StatusBarState.SHADE_LOCKED); @@ -2806,7 +2807,6 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab final boolean isPostureAllowedForFaceAuth = mConfigFaceAuthSupportedPosture == 0 /* DEVICE_POSTURE_UNKNOWN */ ? true : (mPostureState == mConfigFaceAuthSupportedPosture); - // Only listen if this KeyguardUpdateMonitor belongs to the primary user. There is an // instance of KeyguardUpdateMonitor for each user but KeyguardUpdateMonitor is user-aware. final boolean shouldListen = @@ -2816,11 +2816,11 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab || awakeKeyguard || shouldListenForFaceAssistant || isUdfpsFingerDown - || mUdfpsBouncerShowing) + || mAlternateBouncerShowing) && !mSwitchingUser && !faceDisabledForUser && userNotTrustedOrDetectionIsNeeded && !mKeyguardGoingAway && biometricEnabledForUser && faceAuthAllowedOrDetectionIsNeeded && mIsPrimaryUser - && (!mSecureCameraLaunched || mOccludingAppRequestingFace) + && (!mSecureCameraLaunched || mAlternateBouncerShowing) && faceAndFpNotAuthenticated && !mGoingToSleep && isPostureAllowedForFaceAuth; @@ -2831,6 +2831,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab System.currentTimeMillis(), user, shouldListen, + mAlternateBouncerShowing, mAuthInterruptActive, biometricEnabledForUser, mPrimaryBouncerFullyShown, @@ -2848,7 +2849,6 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab mSecureCameraLaunched, supportsDetect, mSwitchingUser, - mUdfpsBouncerShowing, isUdfpsFingerDown, userNotTrustedOrDetectionIsNeeded)); @@ -3942,7 +3942,7 @@ public class KeyguardUpdateMonitor implements TrustManager.TrustListener, Dumpab pw.println(" mPrimaryBouncerIsOrWillBeShowing=" + mPrimaryBouncerIsOrWillBeShowing); pw.println(" mStatusBarState=" + StatusBarState.toString(mStatusBarState)); - pw.println(" mUdfpsBouncerShowing=" + mUdfpsBouncerShowing); + pw.println(" mAlternateBouncerShowing=" + mAlternateBouncerShowing); } else if (isSfpsSupported()) { pw.println(" sfpsEnrolled=" + isSfpsEnrolled()); pw.println(" shouldListenForSfps=" + shouldListenForFingerprint(false)); diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManager.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManager.java index fd465710d4ea..39281da09749 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManager.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManager.java @@ -753,7 +753,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb mKeyguardMessageAreaController.setMessage(""); } mBypassController.setAltBouncerShowing(isShowingAlternateBouncer); - mKeyguardUpdateManager.setUdfpsBouncerShowing(isShowingAlternateBouncer); + mKeyguardUpdateManager.setAlternateBouncerShowing(isShowingAlternateBouncer); if (updateScrim) { mCentralSurfaces.updateScrimController(); diff --git a/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java b/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java index 97c6a93299bf..f5083a756d52 100644 --- a/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java +++ b/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java @@ -1730,7 +1730,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { } @Test - public void testShouldListenForFace_whenOccludingAppRequestsFaceAuth_returnsTrue() + public void shouldListenForFace_secureCameraLaunchedButAlternateBouncerIsLaunched_returnsTrue() throws RemoteException { // Face auth should run when the following is true. keyguardNotGoingAway(); @@ -1746,7 +1746,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isFalse(); - occludingAppRequestsFaceAuth(); + alternateBouncerVisible(); mTestableLooper.processAllMessages(); assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isTrue(); @@ -1836,7 +1836,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { } @Test - public void testShouldListenForFace_whenUdfpsBouncerIsShowing_returnsTrue() + public void testShouldListenForFace_whenAlternateBouncerIsShowing_returnsTrue() throws RemoteException { // Preconditions for face auth to run keyguardNotGoingAway(); @@ -1848,13 +1848,13 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { mTestableLooper.processAllMessages(); assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isFalse(); - mKeyguardUpdateMonitor.setUdfpsBouncerShowing(true); + mKeyguardUpdateMonitor.setAlternateBouncerShowing(true); assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isTrue(); } @Test - public void testShouldListenForFace_udfpsBouncerIsShowingButDeviceGoingToSleep_returnsFalse() + public void testShouldListenForFace_alternateBouncerShowingButDeviceGoingToSleep_returnsFalse() throws RemoteException { // Preconditions for face auth to run keyguardNotGoingAway(); @@ -1864,7 +1864,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { biometricsEnabledForCurrentUser(); userNotCurrentlySwitching(); deviceNotGoingToSleep(); - mKeyguardUpdateMonitor.setUdfpsBouncerShowing(true); + alternateBouncerVisible(); mTestableLooper.processAllMessages(); assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isTrue(); @@ -1874,6 +1874,10 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isFalse(); } + private void alternateBouncerVisible() { + mKeyguardUpdateMonitor.setAlternateBouncerShowing(true); + } + @Test public void testShouldListenForFace_whenFaceIsLockedOut_returnsTrue() throws RemoteException { @@ -1884,7 +1888,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { biometricsNotDisabledThroughDevicePolicyManager(); biometricsEnabledForCurrentUser(); userNotCurrentlySwitching(); - mKeyguardUpdateMonitor.setUdfpsBouncerShowing(true); + mKeyguardUpdateMonitor.setAlternateBouncerShowing(true); mTestableLooper.processAllMessages(); assertThat(mKeyguardUpdateMonitor.shouldListenForFace()).isTrue(); |