From 6e745242dad8b514edd16ff2f593f193dc5074cf Mon Sep 17 00:00:00 2001 From: Austin Delgado Date: Thu, 4 May 2023 14:09:15 -0700 Subject: [PATCH] Fix going back after biometric enrollment in SUW Bug: 280530120 Test: atest BiometricEnrollActivityTest Test: tested manually by swiping back after enrollments Change-Id: I305dd0904f86e2fc77f02047de8f3fc2c54621cb --- .../settings/biometrics/BiometricEnrollActivity.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/com/android/settings/biometrics/BiometricEnrollActivity.java b/src/com/android/settings/biometrics/BiometricEnrollActivity.java index 4d05946f18c..ebbe2e85bcb 100644 --- a/src/com/android/settings/biometrics/BiometricEnrollActivity.java +++ b/src/com/android/settings/biometrics/BiometricEnrollActivity.java @@ -326,8 +326,8 @@ public class BiometricEnrollActivity extends InstrumentedActivity { } } - boolean canUseFace = mIsFaceEnrollable; - boolean canUseFingerprint = mIsFingerprintEnrollable; + boolean canUseFace = mHasFeatureFace; + boolean canUseFingerprint = mHasFeatureFingerprint; if (mParentalOptionsRequired) { if (mParentalOptions == null) { throw new IllegalStateException("consent options required, but not set"); @@ -345,10 +345,12 @@ public class BiometricEnrollActivity extends InstrumentedActivity { } else if (canUseFace || canUseFingerprint) { if (mGkPwHandle == null) { setOrConfirmCredentialsNow(); - } else if (canUseFingerprint) { + } else if (canUseFingerprint && mIsFingerprintEnrollable) { launchFingerprintOnlyEnroll(); - } else { + } else if (canUseFace && mIsFaceEnrollable) { launchFaceOnlyEnroll(); + } else { + setOrConfirmCredentialsNow(); } } else { // no modalities available if (mParentalOptionsRequired) {