Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 45bab9dd authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Do not show 2nd FingerprintErrorDialog" into tm-qpr-dev am: 4fd86cc0 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/20120050



Change-Id: I68b6a8afeb51b04b1c48cf7e4c94e6e9ff5f346e
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 0b79b2ed 4fd86cc0

src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrolling.java

+6 −3
Original line number Diff line number Diff line
@@ -168,7 +168,8 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling { 
    private Vibrator mVibrator;

    private boolean mIsSetupWizard;

    private boolean mIsOrientationChanged;

    private boolean mIsCanceled;

    @VisibleForTesting

    boolean mIsCanceled;

    private AccessibilityManager mAccessibilityManager;

    private boolean mIsAccessibilityEnabled;

    private LottieAnimationView mIllustrationLottie;
@@ -196,7 +197,7 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling { 


    @Override

    public void onWindowFocusChanged(boolean hasFocus) {

        if (hasFocus) {

        if (hasFocus || mIsCanceled) {

            return;

        }
@@ -411,8 +412,10 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling { 


    @VisibleForTesting

    void onCancelEnrollment(@IdRes int errorMsgId) {

        FingerprintErrorDialog.showErrorDialog(this, errorMsgId);

        // showErrorDialog() will cause onWindowFocusChanged(false), set mIsCanceled to false

        // before showErrorDialog() to prevent that another error dialog is triggered again.

        mIsCanceled = true;

        FingerprintErrorDialog.showErrorDialog(this, errorMsgId);

        mIsOrientationChanged = false;

        cancelEnrollment();

        stopIconAnimation();

src/com/android/settings/biometrics/fingerprint/FingerprintErrorDialog.java

+1 −0
Original line number Diff line number Diff line
@@ -72,6 +72,7 @@ public class FingerprintErrorDialog extends InstrumentedDialogFragment { 
                                    dialog.dismiss();

                                    Activity activity = getActivity();

                                    Intent intent = activity.getIntent();

                                    intent.addFlags(Intent.FLAG_ACTIVITY_FORWARD_RESULT);

                                    intent.putExtra(KEY_STATE_CANCELED, false);

                                    activity.startActivity(intent);

                                    activity.finish();

tests/robotests/src/com/android/settings/biometrics/fingerprint/FingerprintEnrollEnrollingTest.java

+10 −0
Original line number Diff line number Diff line
@@ -150,6 +150,16 @@ public class FingerprintEnrollEnrollingTest { 
        verify(mActivity, never()).onCancelEnrollment(anyInt());

    }



    @Test

    public void fingerprintUdfpsOverlayEnrollment_loseFocusWithCancelFlag_shouldNotCancelAgain() {

        initializeActivityFor(TYPE_UDFPS_OPTICAL);



        mActivity.mIsCanceled = true;

        mActivity.onWindowFocusChanged(true);



        verify(mActivity, never()).onCancelEnrollment(anyInt());

    }



    @Test

    public void fingerprintSfpsEnroll_PlaysAllAnimationsAssetsCorrectly() {

        initializeActivityFor(TYPE_POWER_BUTTON);