From 440afb886854d2024194f2b46a993aa473c8f960 Mon Sep 17 00:00:00 2001 From: Kevin Chyn Date: Tue, 29 Jan 2019 17:33:57 -0800 Subject: [PATCH] ConfirmDeviceCredential should default to setRequireConfirmation(true) If the value is unset, we should maintain the same default behavior as BiometricPrompt, which is to dfeault to requiring confirmation. Fixes: 123598496 Test: manual Change-Id: Icaf27d184e0aea5eee077b8ec14bca27fd0ddad3 --- src/com/android/settings/password/BiometricFragment.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/com/android/settings/password/BiometricFragment.java b/src/com/android/settings/password/BiometricFragment.java index 3809a4a5b48..7a685ab535a 100644 --- a/src/com/android/settings/password/BiometricFragment.java +++ b/src/com/android/settings/password/BiometricFragment.java @@ -125,7 +125,8 @@ public class BiometricFragment extends InstrumentedFragment { .setUseDefaultTitle() // use default title if title is null/empty .setSubtitle(mBundle.getString(BiometricPrompt.KEY_SUBTITLE)) .setDescription(mBundle.getString(BiometricPrompt.KEY_DESCRIPTION)) - .setRequireConfirmation(mBundle.getBoolean(BiometricPrompt.KEY_REQUIRE_CONFIRMATION)) + .setRequireConfirmation( + mBundle.getBoolean(BiometricPrompt.KEY_REQUIRE_CONFIRMATION, true)) .setNegativeButton(getResources().getString( R.string.confirm_device_credential_use_alternate_method), mClientExecutor, mNegativeButtonListener)