Merge "[DO NOT MERGE] Do not clear calling identify when using BiometricPrompt... (c8347811) · Commits · e / os / android_frameworks_base

core/java/android/hardware/biometrics/BiometricPrompt.java

+44 −11

Original line number	Diff line number	Diff line
		@@ -407,6 +407,31 @@ public class BiometricPrompt implements BiometricAuthenticator, BiometricConstan
		return this;
		}

		/**
		* Flag to decide if authentication should ignore enrollment state.
		* Defaults to false (not ignoring enrollment state)
		* @param ignoreEnrollmentState
		* @return This builder.
		* @hide
		*/
		@NonNull
		public Builder setIgnoreEnrollmentState(boolean ignoreEnrollmentState) {
		mPromptInfo.setIgnoreEnrollmentState(ignoreEnrollmentState);
		return this;
		}

		/**
		* Set if BiometricPrompt is being used by the legacy fingerprint manager API.
		* @param sensorId sensor id
		* @return This builder.
		* @hide
		*/
		@NonNull
		public Builder setIsForLegacyFingerprintManager(int sensorId) {
		mPromptInfo.setIsForLegacyFingerprintManager(sensorId);
		return this;
		}

		/**
		* Creates a {@link BiometricPrompt}.
		*
		@@ -841,26 +866,34 @@ public class BiometricPrompt implements BiometricAuthenticator, BiometricConstan
		@NonNull @CallbackExecutor Executor executor,
		@NonNull AuthenticationCallback callback,
		int userId) {
		authenticateUserForOperation(cancel, executor, callback, userId, 0 /* operationId */);
		if (cancel == null) {
		throw new IllegalArgumentException("Must supply a cancellation signal");
		}
		if (executor == null) {
		throw new IllegalArgumentException("Must supply an executor");
		}
		if (callback == null) {
		throw new IllegalArgumentException("Must supply a callback");
		}

		authenticateInternal(0 /* operationId */, cancel, executor, callback, userId);
		}

		/**
		* Authenticates for the given user and keystore operation.
		* Authenticates for the given keystore operation.
		*
		* @param cancel An object that can be used to cancel authentication
		* @param executor An executor to handle callback events
		* @param callback An object to receive authentication events
		* @param userId The user to authenticate
		* @param operationId The keystore operation associated with authentication
		*
		* @hide
		*/
		@RequiresPermission(USE_BIOMETRIC_INTERNAL)
		public void authenticateUserForOperation(
		@RequiresPermission(USE_BIOMETRIC)
		public void authenticateForOperation(
		@NonNull CancellationSignal cancel,
		@NonNull @CallbackExecutor Executor executor,
		@NonNull AuthenticationCallback callback,
		int userId,
		long operationId) {
		if (cancel == null) {
		throw new IllegalArgumentException("Must supply a cancellation signal");
		@@ -871,7 +904,8 @@ public class BiometricPrompt implements BiometricAuthenticator, BiometricConstan
		if (callback == null) {
		throw new IllegalArgumentException("Must supply a callback");
		}
		authenticateInternal(operationId, cancel, executor, callback, userId);

		authenticateInternal(operationId, cancel, executor, callback, mContext.getUserId());
		}

		/**
		@@ -1005,7 +1039,7 @@ public class BiometricPrompt implements BiometricAuthenticator, BiometricConstan
		private void cancelAuthentication() {
		if (mService != null) {
		try {
		mService.cancelAuthentication(mToken, mContext.getOpPackageName());
		mService.cancelAuthentication(mToken, mContext.getPackageName());
		} catch (RemoteException e) {
		Log.e(TAG, "Unable to cancel authentication", e);
		}
		@@ -1065,9 +1099,8 @@ public class BiometricPrompt implements BiometricAuthenticator, BiometricConstan
		promptInfo = mPromptInfo;
		}

		mService.authenticate(mToken, operationId, userId, mBiometricServiceReceiver,
		mContext.getOpPackageName(), promptInfo);

		mService.authenticate(mToken, operationId, userId,
		mBiometricServiceReceiver, mContext.getPackageName(), promptInfo);
		} catch (RemoteException e) {
		Log.e(TAG, "Remote exception while authenticating", e);
		mExecutor.execute(() -> callback.onAuthenticationError(

core/java/android/hardware/biometrics/ITestSessionCallback.aidl

+1 −1

Original line number	Diff line number	Diff line
		@@ -19,7 +19,7 @@ package android.hardware.biometrics;
		* ITestSession callback for FingerprintManager and BiometricManager.
		* @hide
		*/
		interface ITestSessionCallback {
		oneway interface ITestSessionCallback {
		void onCleanupStarted(int userId);
		void onCleanupFinished(int userId);
		}

core/java/android/hardware/biometrics/PromptInfo.java

+31 −2

Original line number	Diff line number	Diff line
		@@ -45,6 +45,8 @@ public class PromptInfo implements Parcelable {
		private boolean mReceiveSystemEvents;
		@NonNull private List<Integer> mAllowedSensorIds = new ArrayList<>();
		private boolean mAllowBackgroundAuthentication;
		private boolean mIgnoreEnrollmentState;
		private boolean mIsForLegacyFingerprintManager = false;

		public PromptInfo() {

		@@ -66,6 +68,8 @@ public class PromptInfo implements Parcelable {
		mReceiveSystemEvents = in.readBoolean();
		mAllowedSensorIds = in.readArrayList(Integer.class.getClassLoader());
		mAllowBackgroundAuthentication = in.readBoolean();
		mIgnoreEnrollmentState = in.readBoolean();
		mIsForLegacyFingerprintManager = in.readBoolean();
		}

		public static final Creator<PromptInfo> CREATOR = new Creator<PromptInfo>() {
		@@ -102,10 +106,16 @@ public class PromptInfo implements Parcelable {
		dest.writeBoolean(mReceiveSystemEvents);
		dest.writeList(mAllowedSensorIds);
		dest.writeBoolean(mAllowBackgroundAuthentication);
		dest.writeBoolean(mIgnoreEnrollmentState);
		dest.writeBoolean(mIsForLegacyFingerprintManager);
		}

		public boolean containsTestConfigurations() {
		if (!mAllowedSensorIds.isEmpty()) {
		if (mIsForLegacyFingerprintManager
		&& mAllowedSensorIds.size() == 1
		&& !mAllowBackgroundAuthentication) {
		return false;
		} else if (!mAllowedSensorIds.isEmpty()) {
		return true;
		} else if (mAllowBackgroundAuthentication) {
		return true;
		@@ -185,13 +195,24 @@ public class PromptInfo implements Parcelable {
		}

		public void setAllowedSensorIds(@NonNull List<Integer> sensorIds) {
		mAllowedSensorIds = sensorIds;
		mAllowedSensorIds.clear();
		mAllowedSensorIds.addAll(sensorIds);
		}

		public void setAllowBackgroundAuthentication(boolean allow) {
		mAllowBackgroundAuthentication = allow;
		}

		public void setIgnoreEnrollmentState(boolean ignoreEnrollmentState) {
		mIgnoreEnrollmentState = ignoreEnrollmentState;
		}

		public void setIsForLegacyFingerprintManager(int sensorId) {
		mIsForLegacyFingerprintManager = true;
		mAllowedSensorIds.clear();
		mAllowedSensorIds.add(sensorId);
		}

		// Getters

		public CharSequence getTitle() {
		@@ -261,4 +282,12 @@ public class PromptInfo implements Parcelable {
		public boolean isAllowBackgroundAuthentication() {
		return mAllowBackgroundAuthentication;
		}

		public boolean isIgnoreEnrollmentState() {
		return mIgnoreEnrollmentState;
		}

		public boolean isForLegacyFingerprintManager() {
		return mIsForLegacyFingerprintManager;
		}
		}

packages/SystemUI/src/com/android/systemui/biometrics/AuthController.java

+7 −3

Original line number	Diff line number	Diff line
		@@ -118,7 +118,7 @@ public class AuthController extends SystemUI implements CommandQueue.Callbacks,
		private class BiometricTaskStackListener extends TaskStackListener {
		@Override
		public void onTaskStackChanged() {
		mHandler.post(AuthController.this::handleTaskStackChanged);
		mHandler.post(AuthController.this::cancelIfOwnerIsNotInForeground);
		}
		}

		@@ -181,7 +181,7 @@ public class AuthController extends SystemUI implements CommandQueue.Callbacks,
		}
		};

		private void handleTaskStackChanged() {
		private void cancelIfOwnerIsNotInForeground() {
		if (mCurrentDialog != null) {
		try {
		final String clientPackage = mCurrentDialog.getOpPackageName();
		@@ -192,7 +192,7 @@ public class AuthController extends SystemUI implements CommandQueue.Callbacks,
		final String topPackage = runningTasks.get(0).topActivity.getPackageName();
		if (!topPackage.contentEquals(clientPackage)
		&& !Utils.isSystem(mContext, clientPackage)) {
		Log.w(TAG, "Evicting client due to: " + topPackage);
		Log.e(TAG, "Evicting client due to: " + topPackage);
		mCurrentDialog.dismissWithoutCallback(true /* animate */);
		mCurrentDialog = null;
		mOrientationListener.disable();
		@@ -721,6 +721,10 @@ public class AuthController extends SystemUI implements CommandQueue.Callbacks,
		mCurrentDialog = newDialog;
		mCurrentDialog.show(mWindowManager, savedState);
		mOrientationListener.enable();

		if (!promptInfo.isAllowBackgroundAuthentication()) {
		mHandler.post(this::cancelIfOwnerIsNotInForeground);
		}
		}

		private void onDialogDismissed(@DismissedReason int reason) {

packages/SystemUI/tests/src/com/android/systemui/biometrics/AuthControllerTest.java

+26 −6

Original line number	Diff line number	Diff line
		@@ -485,16 +485,26 @@ public class AuthControllerTest extends SysuiTestCase {
		mAuthController.mLastBiometricPromptInfo.getAuthenticators());
		}

		@Test
		public void testClientNotified_whenTaskStackChangesDuringShow() throws Exception {
		switchTask("other_package");
		showDialog(new int[] {1} /* sensorIds /, false / credentialAllowed */);

		waitForIdleSync();

		assertNull(mAuthController.mCurrentDialog);
		assertNull(mAuthController.mReceiver);
		verify(mDialog1).dismissWithoutCallback(true /* animate */);
		verify(mReceiver).onDialogDismissed(
		eq(BiometricPrompt.DISMISSED_REASON_USER_CANCEL),
		eq(null) /* credentialAttestation */);
		}

		@Test
		public void testClientNotified_whenTaskStackChangesDuringAuthentication() throws Exception {
		showDialog(new int[] {1} /* sensorIds /, false / credentialAllowed */);

		List<ActivityManager.RunningTaskInfo> tasks = new ArrayList<>();
		ActivityManager.RunningTaskInfo taskInfo = mock(ActivityManager.RunningTaskInfo.class);
		taskInfo.topActivity = mock(ComponentName.class);
		when(taskInfo.topActivity.getPackageName()).thenReturn("other_package");
		tasks.add(taskInfo);
		when(mActivityTaskManager.getTasks(anyInt())).thenReturn(tasks);
		switchTask("other_package");

		mAuthController.mTaskStackListener.onTaskStackChanged();
		waitForIdleSync();
		@@ -570,6 +580,16 @@ public class AuthControllerTest extends SysuiTestCase {
		BIOMETRIC_MULTI_SENSOR_FACE_THEN_FINGERPRINT);
		}

		private void switchTask(String packageName) {
		final List<ActivityManager.RunningTaskInfo> tasks = new ArrayList<>();
		final ActivityManager.RunningTaskInfo taskInfo =
		mock(ActivityManager.RunningTaskInfo.class);
		taskInfo.topActivity = mock(ComponentName.class);
		when(taskInfo.topActivity.getPackageName()).thenReturn(packageName);
		tasks.add(taskInfo);
		when(mActivityTaskManager.getTasks(anyInt())).thenReturn(tasks);
		}

		private PromptInfo createTestPromptInfo() {
		PromptInfo promptInfo = new PromptInfo();