From 47e8ddbdb9cd73fc103fa59d5be974eebdf0b7bd Mon Sep 17 00:00:00 2001 From: Joe Bolinger Date: Wed, 30 Jun 2021 14:09:44 -0700 Subject: [PATCH] Prevent finishing early on single sensor devices. Fixes regression from commit c358adad5680d3a8416969e7c739195d995ee5da. Fix: 192420564 Fix: 188847063 Test: manual enroll face & fingerprint via setup wizard (on fp, face, & fp+face device variants) Change-Id: If7027e4d0ddf9d73daf91f174f237e4eb1621ea7 --- .../biometrics/BiometricEnrollActivity.java | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/src/com/android/settings/biometrics/BiometricEnrollActivity.java b/src/com/android/settings/biometrics/BiometricEnrollActivity.java index 3b8f25507e9..26d8a45c5c9 100644 --- a/src/com/android/settings/biometrics/BiometricEnrollActivity.java +++ b/src/com/android/settings/biometrics/BiometricEnrollActivity.java @@ -242,6 +242,7 @@ public class BiometricEnrollActivity extends InstrumentedActivity { // This will need to be updated if the device has sensors other than BIOMETRIC_STRONG if (!setupWizard && authenticators == BiometricManager.Authenticators.DEVICE_CREDENTIAL) { launchCredentialOnlyEnroll(); + finish(); } else if (canUseFace && canUseFingerprint) { if (mParentalOptionsRequired && mGkPwHandle != null) { launchFaceAndFingerprintEnroll(); @@ -405,22 +406,6 @@ public class BiometricEnrollActivity extends InstrumentedActivity { super.onApplyThemeResource(theme, newResid, first); } - @Override - protected void onStop() { - super.onStop(); - - if (mConfirmingCredentials - || mParentalOptionsRequired - || mMultiBiometricEnrollHelper != null) { - return; - } - - if (!isChangingConfigurations()) { - Log.d(TAG, "Finishing in onStop"); - finish(); - } - } - private void setOrConfirmCredentialsNow() { if (!mConfirmingCredentials) { mConfirmingCredentials = true;