From 82a7c8e59bdf7a11d4a5e9ef26f69918083ea959 Mon Sep 17 00:00:00 2001 From: Joe Bolinger Date: Wed, 19 Jan 2022 12:11:33 -0800 Subject: [PATCH] Remove useless type. Replace with standard jvm type. Bug: 204585936 Test: Builds Change-Id: Icd0be731236d417d5c3675982c6ee0a035f4240f --- .../biometrics/sensors/AcquisitionClient.java | 4 +++- .../sensors/AuthenticationClient.java | 3 ++- .../biometrics/sensors/EnrollClient.java | 3 ++- .../sensors/GenerateChallengeClient.java | 4 +++- .../biometrics/sensors/HalClientMonitor.java | 17 ++++--------- .../sensors/InternalCleanupClient.java | 7 +++--- .../sensors/InternalEnumerateClient.java | 3 ++- .../sensors/InvalidationClient.java | 3 ++- .../biometrics/sensors/RemovalClient.java | 3 ++- .../sensors/RevokeChallengeClient.java | 4 +++- .../biometrics/sensors/StartUserClient.java | 4 +++- .../biometrics/sensors/StopUserClient.java | 4 +++- .../face/aidl/FaceAuthenticationClient.java | 3 ++- .../sensors/face/aidl/FaceDetectClient.java | 4 +++- .../sensors/face/aidl/FaceEnrollClient.java | 3 ++- .../aidl/FaceGenerateChallengeClient.java | 4 +++- .../aidl/FaceGetAuthenticatorIdClient.java | 3 ++- .../face/aidl/FaceGetFeatureClient.java | 3 ++- .../face/aidl/FaceInternalCleanupClient.java | 7 +++--- .../aidl/FaceInternalEnumerateClient.java | 3 ++- .../face/aidl/FaceInvalidationClient.java | 3 ++- .../sensors/face/aidl/FaceRemovalClient.java | 3 ++- .../face/aidl/FaceResetLockoutClient.java | 4 +++- .../face/aidl/FaceRevokeChallengeClient.java | 4 +++- .../face/aidl/FaceSetFeatureClient.java | 4 +++- .../face/aidl/FaceStartUserClient.java | 4 +++- .../sensors/face/aidl/FaceStopUserClient.java | 4 +++- .../biometrics/sensors/face/aidl/Sensor.java | 6 ++--- .../biometrics/sensors/face/hidl/Face10.java | 4 ++-- .../face/hidl/FaceAuthenticationClient.java | 3 ++- .../sensors/face/hidl/FaceEnrollClient.java | 3 ++- .../hidl/FaceGenerateChallengeClient.java | 3 ++- .../face/hidl/FaceGetFeatureClient.java | 4 +++- .../face/hidl/FaceInternalCleanupClient.java | 7 +++--- .../hidl/FaceInternalEnumerateClient.java | 3 ++- .../sensors/face/hidl/FaceRemovalClient.java | 3 ++- .../face/hidl/FaceResetLockoutClient.java | 3 ++- .../face/hidl/FaceRevokeChallengeClient.java | 4 +++- .../face/hidl/FaceSetFeatureClient.java | 3 ++- .../face/hidl/FaceUpdateActiveUserClient.java | 3 ++- .../aidl/FingerprintAuthenticationClient.java | 3 ++- .../aidl/FingerprintDetectClient.java | 4 +++- .../aidl/FingerprintEnrollClient.java | 4 +++- .../FingerprintGenerateChallengeClient.java | 4 +++- .../FingerprintGetAuthenticatorIdClient.java | 3 ++- .../FingerprintInternalCleanupClient.java | 7 +++--- .../FingerprintInternalEnumerateClient.java | 3 ++- .../aidl/FingerprintInvalidationClient.java | 3 ++- .../aidl/FingerprintRemovalClient.java | 3 ++- .../aidl/FingerprintResetLockoutClient.java | 4 +++- .../FingerprintRevokeChallengeClient.java | 4 +++- .../aidl/FingerprintStartUserClient.java | 4 +++- .../aidl/FingerprintStopUserClient.java | 4 +++- .../sensors/fingerprint/aidl/Sensor.java | 6 ++--- .../fingerprint/hidl/Fingerprint21.java | 4 ++-- .../hidl/FingerprintAuthenticationClient.java | 3 ++- .../hidl/FingerprintDetectClient.java | 3 ++- .../hidl/FingerprintEnrollClient.java | 4 +++- .../FingerprintGenerateChallengeClient.java | 4 +++- .../FingerprintInternalCleanupClient.java | 7 +++--- .../FingerprintInternalEnumerateClient.java | 3 ++- .../hidl/FingerprintRemovalClient.java | 3 ++- .../FingerprintRevokeChallengeClient.java | 4 +++- .../FingerprintUpdateActiveUserClient.java | 2 +- .../sensors/AcquisitionClientTest.java | 4 +++- .../sensors/BiometricSchedulerTest.java | 24 ++++++++++--------- .../UserAwareBiometricSchedulerTest.java | 6 +++-- 67 files changed, 186 insertions(+), 105 deletions(-) diff --git a/services/core/java/com/android/server/biometrics/sensors/AcquisitionClient.java b/services/core/java/com/android/server/biometrics/sensors/AcquisitionClient.java index 86d72ba1c06f0..3936d5bff1922 100644 --- a/services/core/java/com/android/server/biometrics/sensors/AcquisitionClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/AcquisitionClient.java @@ -29,6 +29,8 @@ import android.os.VibrationEffect; import android.os.Vibrator; import android.util.Slog; +import java.util.function.Supplier; + /** * Abstract {@link HalClientMonitor} subclass that operations eligible/interested in acquisition * messages should extend. @@ -57,7 +59,7 @@ public abstract class AcquisitionClient extends HalClientMonitor implement */ protected abstract void stopHalOperation(); - public AcquisitionClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public AcquisitionClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int cookie, int sensorId, boolean shouldVibrate, int statsModality, int statsAction, int statsClient) { diff --git a/services/core/java/com/android/server/biometrics/sensors/AuthenticationClient.java b/services/core/java/com/android/server/biometrics/sensors/AuthenticationClient.java index 35a0f575ec977..6e3e8821d116a 100644 --- a/services/core/java/com/android/server/biometrics/sensors/AuthenticationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/AuthenticationClient.java @@ -42,6 +42,7 @@ import com.android.server.biometrics.Utils; import java.util.ArrayList; import java.util.List; +import java.util.function.Supplier; /** * A class to keep track of the authentication state for a given client. @@ -105,7 +106,7 @@ public abstract class AuthenticationClient extends AcquisitionClient */ public abstract boolean wasUserDetected(); - public AuthenticationClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public AuthenticationClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int targetUserId, long operationId, boolean restricted, @NonNull String owner, int cookie, boolean requireConfirmation, int sensorId, boolean isStrongBiometric, diff --git a/services/core/java/com/android/server/biometrics/sensors/EnrollClient.java b/services/core/java/com/android/server/biometrics/sensors/EnrollClient.java index 3b7adc1a61763..74f4931cf2aaa 100644 --- a/services/core/java/com/android/server/biometrics/sensors/EnrollClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/EnrollClient.java @@ -29,6 +29,7 @@ import android.util.Slog; import com.android.server.biometrics.BiometricsProto; import java.util.Arrays; +import java.util.function.Supplier; /** * A class to keep track of the enrollment state for a given client. @@ -49,7 +50,7 @@ public abstract class EnrollClient extends AcquisitionClient implements En */ protected abstract boolean hasReachedEnrollmentLimit(); - public EnrollClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public EnrollClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull byte[] hardwareAuthToken, @NonNull String owner, @NonNull BiometricUtils utils, int timeoutSec, int statsModality, int sensorId, boolean shouldVibrate) { diff --git a/services/core/java/com/android/server/biometrics/sensors/GenerateChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/GenerateChallengeClient.java index 6fb6d08cd602e..9689418b1f1ac 100644 --- a/services/core/java/com/android/server/biometrics/sensors/GenerateChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/GenerateChallengeClient.java @@ -25,11 +25,13 @@ import android.util.Slog; import com.android.server.biometrics.BiometricsProto; +import java.util.function.Supplier; + public abstract class GenerateChallengeClient extends HalClientMonitor { private static final String TAG = "GenerateChallengeClient"; - public GenerateChallengeClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public GenerateChallengeClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, listener, userId, owner, 0 /* cookie */, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/HalClientMonitor.java b/services/core/java/com/android/server/biometrics/sensors/HalClientMonitor.java index c8830f8049a21..603af9c86cddf 100644 --- a/services/core/java/com/android/server/biometrics/sensors/HalClientMonitor.java +++ b/services/core/java/com/android/server/biometrics/sensors/HalClientMonitor.java @@ -22,20 +22,13 @@ import android.content.Context; import android.hardware.biometrics.BiometricsProtoEnums; import android.os.IBinder; +import java.util.function.Supplier; + /** * Abstract {@link BaseClientMonitor} implementation that supports HAL operations. * @param HAL template */ public abstract class HalClientMonitor extends BaseClientMonitor { - /** - * Interface that allows ClientMonitor subclasses to retrieve a fresh instance to the HAL. - */ - public interface LazyDaemon { - /** - * @return A fresh instance to the biometric HAL - */ - T getDaemon(); - } /** * Starts the HAL operation specific to the ClientMonitor subclass. @@ -50,7 +43,7 @@ public abstract class HalClientMonitor extends BaseClientMonitor { public abstract void unableToStart(); @NonNull - protected final LazyDaemon mLazyDaemon; + protected final Supplier mLazyDaemon; /** * @param context system_server context @@ -65,7 +58,7 @@ public abstract class HalClientMonitor extends BaseClientMonitor { * @param statsAction One of {@link BiometricsProtoEnums} ACTION_* constants * @param statsClient One of {@link BiometricsProtoEnums} CLIENT_* constants */ - public HalClientMonitor(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public HalClientMonitor(@NonNull Context context, @NonNull Supplier lazyDaemon, @Nullable IBinder token, @Nullable ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int cookie, int sensorId, int statsModality, int statsAction, int statsClient) { @@ -76,6 +69,6 @@ public abstract class HalClientMonitor extends BaseClientMonitor { @Nullable public T getFreshDaemon() { - return mLazyDaemon.getDaemon(); + return mLazyDaemon.get(); } } diff --git a/services/core/java/com/android/server/biometrics/sensors/InternalCleanupClient.java b/services/core/java/com/android/server/biometrics/sensors/InternalCleanupClient.java index 0636893eabf73..0e6d11ec51825 100644 --- a/services/core/java/com/android/server/biometrics/sensors/InternalCleanupClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/InternalCleanupClient.java @@ -28,6 +28,7 @@ import com.android.server.biometrics.BiometricsProto; import java.util.ArrayList; import java.util.List; import java.util.Map; +import java.util.function.Supplier; /** * Wraps {@link InternalEnumerateClient} and {@link RemovalClient}. Keeps track of all the @@ -99,14 +100,14 @@ public abstract class InternalCleanupClient getEnumerateClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int userId, String owner, + Supplier lazyDaemon, IBinder token, int userId, String owner, List enrolledList, BiometricUtils utils, int sensorId); protected abstract RemovalClient getRemovalClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int biometricId, int userId, String owner, + Supplier lazyDaemon, IBinder token, int biometricId, int userId, String owner, BiometricUtils utils, int sensorId, Map authenticatorIds); - protected InternalCleanupClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + protected InternalCleanupClient(@NonNull Context context, @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, int statsModality, @NonNull List enrolledList, @NonNull BiometricUtils utils, @NonNull Map authenticatorIds) { diff --git a/services/core/java/com/android/server/biometrics/sensors/InternalEnumerateClient.java b/services/core/java/com/android/server/biometrics/sensors/InternalEnumerateClient.java index 05ea19a3aa14e..5f97f3711a608 100644 --- a/services/core/java/com/android/server/biometrics/sensors/InternalEnumerateClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/InternalEnumerateClient.java @@ -27,6 +27,7 @@ import com.android.server.biometrics.BiometricsProto; import java.util.ArrayList; import java.util.List; +import java.util.function.Supplier; /** * Internal class to help clean up unknown templates in the HAL and Framework @@ -43,7 +44,7 @@ public abstract class InternalEnumerateClient extends HalClientMonitor // List of templates to remove from the HAL private List mUnknownHALTemplates = new ArrayList<>(); - protected InternalEnumerateClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + protected InternalEnumerateClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, @NonNull List enrolledList, @NonNull BiometricUtils utils, int sensorId, int statsModality) { diff --git a/services/core/java/com/android/server/biometrics/sensors/InvalidationClient.java b/services/core/java/com/android/server/biometrics/sensors/InvalidationClient.java index ee6bb0f0886ae..697d77cfbee67 100644 --- a/services/core/java/com/android/server/biometrics/sensors/InvalidationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/InvalidationClient.java @@ -27,6 +27,7 @@ import android.util.Slog; import com.android.server.biometrics.BiometricsProto; import java.util.Map; +import java.util.function.Supplier; /** * ClientMonitor subclass for requesting authenticatorId invalidation. See @@ -40,7 +41,7 @@ public abstract class InvalidationClient mAuthenticatorIds; @NonNull private final IInvalidationCallback mInvalidationCallback; - public InvalidationClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public InvalidationClient(@NonNull Context context, @NonNull Supplier lazyDaemon, int userId, int sensorId, @NonNull Map authenticatorIds, @NonNull IInvalidationCallback callback) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, diff --git a/services/core/java/com/android/server/biometrics/sensors/RemovalClient.java b/services/core/java/com/android/server/biometrics/sensors/RemovalClient.java index e79819b401eac..a0cef94fcf48b 100644 --- a/services/core/java/com/android/server/biometrics/sensors/RemovalClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/RemovalClient.java @@ -27,6 +27,7 @@ import android.util.Slog; import com.android.server.biometrics.BiometricsProto; import java.util.Map; +import java.util.function.Supplier; /** * A class to keep track of the remove state for a given client. @@ -40,7 +41,7 @@ public abstract class RemovalClient mAuthenticatorIds; private final boolean mHasEnrollmentsBeforeStarting; - public RemovalClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public RemovalClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, @NonNull Map authenticatorIds, int statsModality) { diff --git a/services/core/java/com/android/server/biometrics/sensors/RevokeChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/RevokeChallengeClient.java index 21a6ddfcde66e..7d8386337eceb 100644 --- a/services/core/java/com/android/server/biometrics/sensors/RevokeChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/RevokeChallengeClient.java @@ -23,9 +23,11 @@ import android.os.IBinder; import com.android.server.biometrics.BiometricsProto; +import java.util.function.Supplier; + public abstract class RevokeChallengeClient extends HalClientMonitor { - public RevokeChallengeClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public RevokeChallengeClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, null /* listener */, userId, owner, 0 /* cookie */, sensorId, BiometricsProtoEnums.MODALITY_UNKNOWN, diff --git a/services/core/java/com/android/server/biometrics/sensors/StartUserClient.java b/services/core/java/com/android/server/biometrics/sensors/StartUserClient.java index 3d6932639c1c2..a74fe4f63cb62 100644 --- a/services/core/java/com/android/server/biometrics/sensors/StartUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/StartUserClient.java @@ -25,6 +25,8 @@ import android.os.IBinder; import com.android.internal.annotations.VisibleForTesting; import com.android.server.biometrics.BiometricsProto; +import java.util.function.Supplier; + /** * Abstract class for starting a new user. * @param Interface to request a new user. @@ -43,7 +45,7 @@ public abstract class StartUserClient extends HalClientMonitor { @NonNull @VisibleForTesting protected final UserStartedCallback mUserStartedCallback; - public StartUserClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public StartUserClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStartedCallback callback) { super(context, lazyDaemon, token, null /* listener */, userId, context.getOpPackageName(), diff --git a/services/core/java/com/android/server/biometrics/sensors/StopUserClient.java b/services/core/java/com/android/server/biometrics/sensors/StopUserClient.java index 1f6e1e95050d4..3eafbb8ea9d72 100644 --- a/services/core/java/com/android/server/biometrics/sensors/StopUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/StopUserClient.java @@ -25,6 +25,8 @@ import android.os.IBinder; import com.android.internal.annotations.VisibleForTesting; import com.android.server.biometrics.BiometricsProto; +import java.util.function.Supplier; + /** * Abstract class for stopping a user. * @param Interface for stopping the user. @@ -43,7 +45,7 @@ public abstract class StopUserClient extends HalClientMonitor { getCallback().onClientFinished(this, true /* success */); } - public StopUserClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public StopUserClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStoppedCallback callback) { super(context, lazyDaemon, token, null /* listener */, userId, context.getOpPackageName(), diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceAuthenticationClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceAuthenticationClient.java index dc21a04fb4464..f9a0efc4b1014 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceAuthenticationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceAuthenticationClient.java @@ -48,6 +48,7 @@ import com.android.server.biometrics.sensors.LockoutTracker; import com.android.server.biometrics.sensors.face.UsageStats; import java.util.ArrayList; +import java.util.function.Supplier; /** * Face-specific authentication client for the {@link IFace} AIDL HAL interface. @@ -69,7 +70,7 @@ class FaceAuthenticationClient extends AuthenticationClient implements @FaceManager.FaceAcquired private int mLastAcquire = FaceManager.FACE_ACQUIRED_UNKNOWN; FaceAuthenticationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int targetUserId, long operationId, boolean restricted, String owner, int cookie, boolean requireConfirmation, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceDetectClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceDetectClient.java index 72a20db077dd7..84ce8df08ef27 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceDetectClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceDetectClient.java @@ -34,6 +34,8 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.DetectionConsumer; +import java.util.function.Supplier; + /** * Performs face detection without exposing any matching information (e.g. accept/reject have the * same haptic, lockout counter is not increased). @@ -46,7 +48,7 @@ public class FaceDetectClient extends AcquisitionClient implements Det @Nullable private ICancellationSignal mCancellationSignal; @Nullable private SensorPrivacyManager mSensorPrivacyManager; - public FaceDetectClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public FaceDetectClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, boolean isStrongBiometric, int statsClient) { diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceEnrollClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceEnrollClient.java index 5c57dbbffedbf..b1be9fbe443dd 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceEnrollClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceEnrollClient.java @@ -51,6 +51,7 @@ import com.android.server.biometrics.sensors.face.FaceUtils; import java.io.IOException; import java.util.ArrayList; import java.util.List; +import java.util.function.Supplier; /** * Face-specific enroll client for the {@link IFace} AIDL HAL interface. @@ -82,7 +83,7 @@ public class FaceEnrollClient extends EnrollClient { } }; - FaceEnrollClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceEnrollClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull byte[] hardwareAuthToken, @NonNull String opPackageName, long requestId, @NonNull BiometricUtils utils, @NonNull int[] disabledFeatures, int timeoutSec, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGenerateChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGenerateChallengeClient.java index 7cdeebb87fece..aebbc5348f95b 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGenerateChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGenerateChallengeClient.java @@ -27,6 +27,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.GenerateChallengeClient; +import java.util.function.Supplier; + /** * Face-specific generateChallenge client for the {@link IFace} AIDL HAL interface. */ @@ -34,7 +36,7 @@ public class FaceGenerateChallengeClient extends GenerateChallengeClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, listener, userId, owner, sensorId); diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetAuthenticatorIdClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetAuthenticatorIdClient.java index 584b58cdd7c7b..98aed7e6d6343 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetAuthenticatorIdClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetAuthenticatorIdClient.java @@ -28,6 +28,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.HalClientMonitor; import java.util.Map; +import java.util.function.Supplier; class FaceGetAuthenticatorIdClient extends HalClientMonitor { @@ -35,7 +36,7 @@ class FaceGetAuthenticatorIdClient extends HalClientMonitor { private final Map mAuthenticatorIds; - FaceGetAuthenticatorIdClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceGetAuthenticatorIdClient(@NonNull Context context, @NonNull Supplier lazyDaemon, int userId, @NonNull String opPackageName, int sensorId, Map authenticatorIds) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, opPackageName, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetFeatureClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetFeatureClient.java index acf5720cd0cf5..3401587caa341 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetFeatureClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceGetFeatureClient.java @@ -36,6 +36,7 @@ import com.android.server.biometrics.sensors.HalClientMonitor; import java.util.HashMap; import java.util.Map; +import java.util.function.Supplier; /** * Face-specific get feature client for the {@link IFace} AIDL HAL interface. @@ -46,7 +47,7 @@ public class FaceGetFeatureClient extends HalClientMonitor implements private final int mUserId; - FaceGetFeatureClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceGetFeatureClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @Nullable ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, listener, userId, owner, 0 /* cookie */, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalCleanupClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalCleanupClient.java index c6696aed6520d..19095c59896f0 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalCleanupClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalCleanupClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.RemovalClient; import java.util.List; import java.util.Map; +import java.util.function.Supplier; /** * Face-specific internal cleanup client for the {@link IFace} AIDL HAL interface. @@ -38,7 +39,7 @@ import java.util.Map; class FaceInternalCleanupClient extends InternalCleanupClient { FaceInternalCleanupClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, @NonNull String owner, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, @NonNull List enrolledList, @NonNull BiometricUtils utils, @NonNull Map authenticatorIds) { super(context, lazyDaemon, userId, owner, sensorId, BiometricsProtoEnums.MODALITY_FACE, @@ -47,7 +48,7 @@ class FaceInternalCleanupClient extends InternalCleanupClient { @Override protected InternalEnumerateClient getEnumerateClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int userId, String owner, + Supplier lazyDaemon, IBinder token, int userId, String owner, List enrolledList, BiometricUtils utils, int sensorId) { return new FaceInternalEnumerateClient(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId); @@ -55,7 +56,7 @@ class FaceInternalCleanupClient extends InternalCleanupClient { @Override protected RemovalClient getRemovalClient(Context context, - LazyDaemon lazyDaemon, IBinder token, + Supplier lazyDaemon, IBinder token, int biometricId, int userId, String owner, BiometricUtils utils, int sensorId, Map authenticatorIds) { // Internal remove does not need to send results to anyone. Cleanup (enumerate + remove) diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalEnumerateClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalEnumerateClient.java index 0ece884ecff81..de5f1154fd2b4 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalEnumerateClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInternalEnumerateClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.BiometricUtils; import com.android.server.biometrics.sensors.InternalEnumerateClient; import java.util.List; +import java.util.function.Supplier; /** * Face-specific internal enumerate client for the {@link IFace} AIDL HAL interface. @@ -38,7 +39,7 @@ class FaceInternalEnumerateClient extends InternalEnumerateClient { private static final String TAG = "FaceInternalEnumerateClient"; FaceInternalEnumerateClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, int userId, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, @NonNull List enrolledList, @NonNull BiometricUtils utils, int sensorId) { super(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInvalidationClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInvalidationClient.java index 405e2b23b3af0..416921a0b098b 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInvalidationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceInvalidationClient.java @@ -27,12 +27,13 @@ import android.util.Slog; import com.android.server.biometrics.sensors.InvalidationClient; import java.util.Map; +import java.util.function.Supplier; public class FaceInvalidationClient extends InvalidationClient { private static final String TAG = "FaceInvalidationClient"; public FaceInvalidationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, int sensorId, + @NonNull Supplier lazyDaemon, int userId, int sensorId, @NonNull Map authenticatorIds, @NonNull IInvalidationCallback callback) { super(context, lazyDaemon, userId, sensorId, authenticatorIds, callback); } diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRemovalClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRemovalClient.java index ba678f3b19ae1..1995c378fff90 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRemovalClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRemovalClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.RemovalClient; import java.util.Map; +import java.util.function.Supplier; /** * Face-specific removal client for the {@link IFace} AIDL HAL interface. @@ -40,7 +41,7 @@ class FaceRemovalClient extends RemovalClient { final int[] mBiometricIds; - FaceRemovalClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceRemovalClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int[] biometricIds, int userId, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceResetLockoutClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceResetLockoutClient.java index fd44c5cf4afc5..0a3e7651bac4f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceResetLockoutClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceResetLockoutClient.java @@ -34,6 +34,8 @@ import com.android.server.biometrics.sensors.LockoutCache; import com.android.server.biometrics.sensors.LockoutResetDispatcher; import com.android.server.biometrics.sensors.LockoutTracker; +import java.util.function.Supplier; + /** * Face-specific resetLockout client for the {@link IFace} AIDL HAL interface. * Updates the framework's lockout cache and notifies clients such as Keyguard when lockout is @@ -48,7 +50,7 @@ public class FaceResetLockoutClient extends HalClientMonitor implement private final LockoutResetDispatcher mLockoutResetDispatcher; FaceResetLockoutClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, String owner, int sensorId, + @NonNull Supplier lazyDaemon, int userId, String owner, int sensorId, @NonNull byte[] hardwareAuthToken, @NonNull LockoutCache lockoutTracker, @NonNull LockoutResetDispatcher lockoutResetDispatcher) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, owner, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRevokeChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRevokeChallengeClient.java index 7a69c443411f6..5e902ac480a5f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRevokeChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceRevokeChallengeClient.java @@ -26,6 +26,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.RevokeChallengeClient; +import java.util.function.Supplier; + /** * Face-specific revokeChallenge client for the {@link IFace} AIDL HAL interface. */ @@ -36,7 +38,7 @@ public class FaceRevokeChallengeClient extends RevokeChallengeClient { private final long mChallenge; FaceRevokeChallengeClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, int sensorId, long challenge) { super(context, lazyDaemon, token, userId, owner, sensorId); mChallenge = challenge; diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceSetFeatureClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceSetFeatureClient.java index ee6982abd9ed1..962f900c87fd2 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceSetFeatureClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceSetFeatureClient.java @@ -33,6 +33,8 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.ErrorConsumer; import com.android.server.biometrics.sensors.HalClientMonitor; +import java.util.function.Supplier; + /** * Face-specific get feature client for the {@link IFace} AIDL HAL interface. */ @@ -44,7 +46,7 @@ public class FaceSetFeatureClient extends HalClientMonitor implements private final boolean mEnabled; private final HardwareAuthToken mHardwareAuthToken; - FaceSetFeatureClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceSetFeatureClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, int feature, boolean enabled, byte[] hardwareAuthToken) { diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStartUserClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStartUserClient.java index 4a3da0d929dcb..9b559034b9aff 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStartUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStartUserClient.java @@ -30,12 +30,14 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.StartUserClient; +import java.util.function.Supplier; + public class FaceStartUserClient extends StartUserClient { private static final String TAG = "FaceStartUserClient"; @NonNull private final ISessionCallback mSessionCallback; - public FaceStartUserClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public FaceStartUserClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull ISessionCallback sessionCallback, @NonNull UserStartedCallback callback) { diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStopUserClient.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStopUserClient.java index 88b92359268c9..55da0d284af90 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStopUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/FaceStopUserClient.java @@ -27,10 +27,12 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.StopUserClient; +import java.util.function.Supplier; + public class FaceStopUserClient extends StopUserClient { private static final String TAG = "FaceStopUserClient"; - public FaceStopUserClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + public FaceStopUserClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStoppedCallback callback) { super(context, lazyDaemon, token, userId, sensorId, callback); diff --git a/services/core/java/com/android/server/biometrics/sensors/face/aidl/Sensor.java b/services/core/java/com/android/server/biometrics/sensors/face/aidl/Sensor.java index 206b8f0779e8e..6ae3a9e3a821c 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/aidl/Sensor.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/aidl/Sensor.java @@ -53,7 +53,6 @@ import com.android.server.biometrics.sensors.BaseClientMonitor; import com.android.server.biometrics.sensors.BiometricScheduler; import com.android.server.biometrics.sensors.EnumerateConsumer; import com.android.server.biometrics.sensors.ErrorConsumer; -import com.android.server.biometrics.sensors.HalClientMonitor; import com.android.server.biometrics.sensors.Interruptable; import com.android.server.biometrics.sensors.LockoutCache; import com.android.server.biometrics.sensors.LockoutConsumer; @@ -67,6 +66,7 @@ import com.android.server.biometrics.sensors.face.FaceUtils; import java.util.ArrayList; import java.util.HashMap; import java.util.Map; +import java.util.function.Supplier; /** * Maintains the state of a single sensor within an instance of the {@link IFace} HAL. @@ -84,7 +84,7 @@ public class Sensor { @NonNull private final UserAwareBiometricScheduler mScheduler; @NonNull private final LockoutCache mLockoutCache; @NonNull private final Map mAuthenticatorIds; - @NonNull private final HalClientMonitor.LazyDaemon mLazySession; + @NonNull private final Supplier mLazySession; @Nullable private Session mCurrentSession; static class Session { @@ -545,7 +545,7 @@ public class Sensor { mLazySession = () -> mCurrentSession != null ? mCurrentSession.mSession : null; } - @NonNull HalClientMonitor.LazyDaemon getLazySession() { + @NonNull Supplier getLazySession() { return mLazySession; } diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/Face10.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/Face10.java index 9a52db19ecdad..586abe2d6298e 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/Face10.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/Face10.java @@ -64,7 +64,6 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.EnumerateConsumer; import com.android.server.biometrics.sensors.ErrorConsumer; -import com.android.server.biometrics.sensors.HalClientMonitor; import com.android.server.biometrics.sensors.LockoutResetDispatcher; import com.android.server.biometrics.sensors.LockoutTracker; import com.android.server.biometrics.sensors.PerformanceTracker; @@ -89,6 +88,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.concurrent.atomic.AtomicLong; +import java.util.function.Supplier; /** * Supports a single instance of the {@link android.hardware.biometrics.face.V1_0} or its extended @@ -111,7 +111,7 @@ public class Face10 implements IHwBinder.DeathRecipient, ServiceProvider { @NonNull private final Context mContext; @NonNull private final BiometricScheduler mScheduler; @NonNull private final Handler mHandler; - @NonNull private final HalClientMonitor.LazyDaemon mLazyDaemon; + @NonNull private final Supplier mLazyDaemon; @NonNull private final LockoutHalImpl mLockoutTracker; @NonNull private final UsageStats mUsageStats; @NonNull private final Map mAuthenticatorIds; diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceAuthenticationClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceAuthenticationClient.java index 1e0e7992bf874..9038435c10217 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceAuthenticationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceAuthenticationClient.java @@ -41,6 +41,7 @@ import com.android.server.biometrics.sensors.LockoutTracker; import com.android.server.biometrics.sensors.face.UsageStats; import java.util.ArrayList; +import java.util.function.Supplier; /** * Face-specific authentication client supporting the {@link android.hardware.biometrics.face.V1_0} @@ -61,7 +62,7 @@ class FaceAuthenticationClient extends AuthenticationClient { private SensorPrivacyManager mSensorPrivacyManager; FaceAuthenticationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int targetUserId, long operationId, boolean restricted, String owner, int cookie, boolean requireConfirmation, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceEnrollClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceEnrollClient.java index 8068e14cf0f0c..92f7253779ed8 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceEnrollClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceEnrollClient.java @@ -40,6 +40,7 @@ import com.android.server.biometrics.sensors.EnrollClient; import java.util.ArrayList; import java.util.Arrays; +import java.util.function.Supplier; /** * Face-specific enroll client supporting the {@link android.hardware.biometrics.face.V1_0} HIDL @@ -53,7 +54,7 @@ public class FaceEnrollClient extends EnrollClient { @NonNull private final int[] mEnrollIgnoreList; @NonNull private final int[] mEnrollIgnoreListVendor; - FaceEnrollClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceEnrollClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull byte[] hardwareAuthToken, @NonNull String owner, long requestId, @NonNull BiometricUtils utils, @NonNull int[] disabledFeatures, int timeoutSec, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGenerateChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGenerateChallengeClient.java index e29a1923e47e6..b66ad608b4caf 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGenerateChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGenerateChallengeClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.GenerateChallengeClient; import java.util.ArrayList; import java.util.List; +import java.util.function.Supplier; /** * Face-specific generateChallenge client supporting the @@ -48,7 +49,7 @@ public class FaceGenerateChallengeClient extends GenerateChallengeClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, long now) { super(context, lazyDaemon, token, listener, userId, owner, sensorId); diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGetFeatureClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGetFeatureClient.java index 0a9d96d00eb6f..1b387bf7879af 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGetFeatureClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceGetFeatureClient.java @@ -32,6 +32,8 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.HalClientMonitor; +import java.util.function.Supplier; + /** * Face-specific getFeature client supporting the {@link android.hardware.biometrics.face.V1_0} * HIDL interface. @@ -44,7 +46,7 @@ public class FaceGetFeatureClient extends HalClientMonitor { private final int mFaceId; private boolean mValue; - FaceGetFeatureClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceGetFeatureClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @Nullable ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, int feature, int faceId) { super(context, lazyDaemon, token, listener, userId, owner, 0 /* cookie */, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalCleanupClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalCleanupClient.java index 1e3b92dcbf619..93a2913fbfa16 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalCleanupClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalCleanupClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.RemovalClient; import java.util.List; import java.util.Map; +import java.util.function.Supplier; /** * Face-specific internal cleanup client supporting the @@ -38,7 +39,7 @@ import java.util.Map; class FaceInternalCleanupClient extends InternalCleanupClient { FaceInternalCleanupClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, @NonNull String owner, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, @NonNull List enrolledList, @NonNull BiometricUtils utils, @NonNull Map authenticatorIds) { super(context, lazyDaemon, userId, owner, sensorId, BiometricsProtoEnums.MODALITY_FACE, @@ -47,7 +48,7 @@ class FaceInternalCleanupClient extends InternalCleanupClient getEnumerateClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int userId, String owner, + Supplier lazyDaemon, IBinder token, int userId, String owner, List enrolledList, BiometricUtils utils, int sensorId) { return new FaceInternalEnumerateClient(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId); @@ -55,7 +56,7 @@ class FaceInternalCleanupClient extends InternalCleanupClient getRemovalClient(Context context, - LazyDaemon lazyDaemon, IBinder token, + Supplier lazyDaemon, IBinder token, int biometricId, int userId, String owner, BiometricUtils utils, int sensorId, Map authenticatorIds) { // Internal remove does not need to send results to anyone. Cleanup (enumerate + remove) diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalEnumerateClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalEnumerateClient.java index f2a9afce1ff75..f1788de385657 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalEnumerateClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceInternalEnumerateClient.java @@ -29,6 +29,7 @@ import com.android.server.biometrics.sensors.BiometricUtils; import com.android.server.biometrics.sensors.InternalEnumerateClient; import java.util.List; +import java.util.function.Supplier; /** * Face-specific internal enumerate client supporting the @@ -38,7 +39,7 @@ class FaceInternalEnumerateClient extends InternalEnumerateClient lazyDaemon, @NonNull IBinder token, int userId, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, @NonNull List enrolledList, @NonNull BiometricUtils utils, int sensorId) { super(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRemovalClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRemovalClient.java index 3ae201134debb..cbc23e49f4d8f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRemovalClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRemovalClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.RemovalClient; import java.util.Map; +import java.util.function.Supplier; /** * Face-specific removal client supporting the {@link android.hardware.biometrics.face.V1_0} @@ -40,7 +41,7 @@ class FaceRemovalClient extends RemovalClient { private final int mBiometricId; - FaceRemovalClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceRemovalClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int biometricId, int userId, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, @NonNull Map authenticatorIds) { diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceResetLockoutClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceResetLockoutClient.java index ee01c43a2e732..88e2318b05702 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceResetLockoutClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceResetLockoutClient.java @@ -28,6 +28,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.HalClientMonitor; import java.util.ArrayList; +import java.util.function.Supplier; /** * Face-specific resetLockout client supporting the {@link android.hardware.biometrics.face.V1_0} @@ -40,7 +41,7 @@ public class FaceResetLockoutClient extends HalClientMonitor { private final ArrayList mHardwareAuthToken; FaceResetLockoutClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, String owner, int sensorId, + @NonNull Supplier lazyDaemon, int userId, String owner, int sensorId, @NonNull byte[] hardwareAuthToken) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, owner, 0 /* cookie */, sensorId, BiometricsProtoEnums.MODALITY_UNKNOWN, diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRevokeChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRevokeChallengeClient.java index 5ec7a9819b27d..ab8d16145fab7 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRevokeChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceRevokeChallengeClient.java @@ -25,6 +25,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.RevokeChallengeClient; +import java.util.function.Supplier; + /** * Face-specific revokeChallenge client supporting the {@link android.hardware.biometrics.face.V1_0} * HIDL interface. @@ -34,7 +36,7 @@ public class FaceRevokeChallengeClient extends RevokeChallengeClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, userId, owner, sensorId); } diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceSetFeatureClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceSetFeatureClient.java index ee28f7b0f3047..b2b52e713ec93 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceSetFeatureClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceSetFeatureClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.HalClientMonitor; import java.util.ArrayList; +import java.util.function.Supplier; /** * Face-specific setFeature client supporting the {@link android.hardware.biometrics.face.V1_0} @@ -45,7 +46,7 @@ public class FaceSetFeatureClient extends HalClientMonitor { private final ArrayList mHardwareAuthToken; private final int mFaceId; - FaceSetFeatureClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FaceSetFeatureClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, int feature, boolean enabled, byte[] hardwareAuthToken, int faceId) { diff --git a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceUpdateActiveUserClient.java b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceUpdateActiveUserClient.java index 8ee8ce5220352..04b93274e42b2 100644 --- a/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceUpdateActiveUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/face/hidl/FaceUpdateActiveUserClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.HalClientMonitor; import java.io.File; import java.util.Map; +import java.util.function.Supplier; public class FaceUpdateActiveUserClient extends HalClientMonitor { private static final String TAG = "FaceUpdateActiveUserClient"; @@ -39,7 +40,7 @@ public class FaceUpdateActiveUserClient extends HalClientMonitor mAuthenticatorIds; FaceUpdateActiveUserClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, @NonNull String owner, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, boolean hasEnrolledBiometrics, @NonNull Map authenticatorIds) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, owner, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintAuthenticationClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintAuthenticationClient.java index f3d0121c98eb2..62a8a47da9114 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintAuthenticationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintAuthenticationClient.java @@ -47,6 +47,7 @@ import com.android.server.biometrics.sensors.SensorOverlays; import com.android.server.biometrics.sensors.fingerprint.Udfps; import java.util.ArrayList; +import java.util.function.Supplier; /** * Fingerprint-specific authentication client supporting the @@ -65,7 +66,7 @@ class FingerprintAuthenticationClient extends AuthenticationClient imp private boolean mIsPointerDown; FingerprintAuthenticationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int targetUserId, long operationId, boolean restricted, @NonNull String owner, int cookie, boolean requireConfirmation, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintDetectClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintDetectClient.java index 1f0482db228b3..55960f61d4aeb 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintDetectClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintDetectClient.java @@ -35,6 +35,8 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.DetectionConsumer; import com.android.server.biometrics.sensors.SensorOverlays; +import java.util.function.Supplier; + /** * Performs fingerprint detection without exposing any matching information (e.g. accept/reject * have the same haptic, lockout counter is not increased). @@ -47,7 +49,7 @@ class FingerprintDetectClient extends AcquisitionClient implements Det @NonNull private final SensorOverlays mSensorOverlays; @Nullable private ICancellationSignal mCancellationSignal; - FingerprintDetectClient(@NonNull Context context, @NonNull LazyDaemon lazyDaemon, + FingerprintDetectClient(@NonNull Context context, @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintEnrollClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintEnrollClient.java index 169c3ebec1a74..ee47e9586265b 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintEnrollClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintEnrollClient.java @@ -46,6 +46,8 @@ import com.android.server.biometrics.sensors.fingerprint.FingerprintUtils; import com.android.server.biometrics.sensors.fingerprint.Udfps; import com.android.server.biometrics.sensors.fingerprint.UdfpsHelper; +import java.util.function.Supplier; + class FingerprintEnrollClient extends EnrollClient implements Udfps { private static final String TAG = "FingerprintEnrollClient"; @@ -59,7 +61,7 @@ class FingerprintEnrollClient extends EnrollClient implements Udfps { private boolean mIsPointerDown; FingerprintEnrollClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, long requestId, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull byte[] hardwareAuthToken, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGenerateChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGenerateChallengeClient.java index 4f54f8ade7c00..37131768f6d86 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGenerateChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGenerateChallengeClient.java @@ -27,6 +27,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.GenerateChallengeClient; +import java.util.function.Supplier; + /** * Fingerprint-specific generateChallenge client for the {@link IFingerprint} AIDL HAL interface. */ @@ -34,7 +36,7 @@ class FingerprintGenerateChallengeClient extends GenerateChallengeClient lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId) { diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGetAuthenticatorIdClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGetAuthenticatorIdClient.java index 52bd234fcc5dd..bfb979bc98641 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGetAuthenticatorIdClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintGetAuthenticatorIdClient.java @@ -28,6 +28,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.HalClientMonitor; import java.util.Map; +import java.util.function.Supplier; class FingerprintGetAuthenticatorIdClient extends HalClientMonitor { @@ -36,7 +37,7 @@ class FingerprintGetAuthenticatorIdClient extends HalClientMonitor { private final Map mAuthenticatorIds; FingerprintGetAuthenticatorIdClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, @NonNull String owner, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, Map authenticatorIds) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, owner, 0 /* cookie */, sensorId, BiometricsProtoEnums.MODALITY_FINGERPRINT, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalCleanupClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalCleanupClient.java index 0de3f4f8cce23..a608ba572f571 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalCleanupClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalCleanupClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.fingerprint.FingerprintUtils; import java.util.List; import java.util.Map; +import java.util.function.Supplier; /** * Fingerprint-specific internal cleanup client supporting the @@ -39,7 +40,7 @@ import java.util.Map; class FingerprintInternalCleanupClient extends InternalCleanupClient { FingerprintInternalCleanupClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, @NonNull String owner, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, @NonNull List enrolledList, @NonNull FingerprintUtils utils, @NonNull Map authenticatorIds) { super(context, lazyDaemon, userId, owner, sensorId, @@ -48,7 +49,7 @@ class FingerprintInternalCleanupClient extends InternalCleanupClient getEnumerateClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int userId, String owner, + Supplier lazyDaemon, IBinder token, int userId, String owner, List enrolledList, BiometricUtils utils, int sensorId) { return new FingerprintInternalEnumerateClient(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId); @@ -56,7 +57,7 @@ class FingerprintInternalCleanupClient extends InternalCleanupClient getRemovalClient(Context context, - LazyDaemon lazyDaemon, IBinder token, int biometricId, int userId, + Supplier lazyDaemon, IBinder token, int biometricId, int userId, String owner, BiometricUtils utils, int sensorId, Map authenticatorIds) { return new FingerprintRemovalClient(context, lazyDaemon, token, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalEnumerateClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalEnumerateClient.java index e20544a86d010..e2ec823f00667 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalEnumerateClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInternalEnumerateClient.java @@ -29,6 +29,7 @@ import com.android.server.biometrics.sensors.BiometricUtils; import com.android.server.biometrics.sensors.InternalEnumerateClient; import java.util.List; +import java.util.function.Supplier; /** * Fingerprint-specific internal client supporting the @@ -38,7 +39,7 @@ class FingerprintInternalEnumerateClient extends InternalEnumerateClient lazyDaemon, @NonNull IBinder token, int userId, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, @NonNull List enrolledList, @NonNull BiometricUtils utils, int sensorId) { super(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInvalidationClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInvalidationClient.java index 6cd2ef1aa0e6c..91e3e2a4b8f7f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInvalidationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintInvalidationClient.java @@ -27,12 +27,13 @@ import android.util.Slog; import com.android.server.biometrics.sensors.InvalidationClient; import java.util.Map; +import java.util.function.Supplier; public class FingerprintInvalidationClient extends InvalidationClient { private static final String TAG = "FingerprintInvalidationClient"; public FingerprintInvalidationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, int sensorId, + @NonNull Supplier lazyDaemon, int userId, int sensorId, @NonNull Map authenticatorIds, @NonNull IInvalidationCallback callback) { super(context, lazyDaemon, userId, sensorId, authenticatorIds, callback); } diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRemovalClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRemovalClient.java index 9a9d6ab19eb6e..2cef8570387cf 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRemovalClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRemovalClient.java @@ -31,6 +31,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.RemovalClient; import java.util.Map; +import java.util.function.Supplier; /** * Fingerprint-specific removal client supporting the @@ -42,7 +43,7 @@ class FingerprintRemovalClient extends RemovalClient { private final int[] mBiometricIds; FingerprintRemovalClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @Nullable ClientMonitorCallbackConverter listener, int[] biometricIds, int userId, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, @NonNull Map authenticatorIds) { diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintResetLockoutClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintResetLockoutClient.java index ee8d170af4074..9c7df589d29f2 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintResetLockoutClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintResetLockoutClient.java @@ -34,6 +34,8 @@ import com.android.server.biometrics.sensors.LockoutCache; import com.android.server.biometrics.sensors.LockoutResetDispatcher; import com.android.server.biometrics.sensors.LockoutTracker; +import java.util.function.Supplier; + /** * Fingerprint-specific resetLockout client for the {@link IFingerprint} AIDL HAL interface. * Updates the framework's lockout cache and notifies clients such as Keyguard when lockout is @@ -48,7 +50,7 @@ class FingerprintResetLockoutClient extends HalClientMonitor implement private final LockoutResetDispatcher mLockoutResetDispatcher; FingerprintResetLockoutClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, String owner, int sensorId, + @NonNull Supplier lazyDaemon, int userId, String owner, int sensorId, @NonNull byte[] hardwareAuthToken, @NonNull LockoutCache lockoutTracker, @NonNull LockoutResetDispatcher lockoutResetDispatcher) { super(context, lazyDaemon, null /* token */, null /* listener */, userId, owner, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRevokeChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRevokeChallengeClient.java index 9e6f1bc2ff677..28292010fe748 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRevokeChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintRevokeChallengeClient.java @@ -26,6 +26,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.RevokeChallengeClient; +import java.util.function.Supplier; + /** * Fingerprint-specific revokeChallenge client for the {@link IFingerprint} AIDL HAL interface. */ @@ -36,7 +38,7 @@ class FingerprintRevokeChallengeClient extends RevokeChallengeClient { private final long mChallenge; FingerprintRevokeChallengeClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, int sensorId, long challenge) { super(context, lazyDaemon, token, userId, owner, sensorId); mChallenge = challenge; diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStartUserClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStartUserClient.java index 9f11df6b939b9..ef53df4ca594f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStartUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStartUserClient.java @@ -30,13 +30,15 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.StartUserClient; +import java.util.function.Supplier; + public class FingerprintStartUserClient extends StartUserClient { private static final String TAG = "FingerprintStartUserClient"; @NonNull private final ISessionCallback mSessionCallback; public FingerprintStartUserClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, + @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull ISessionCallback sessionCallback, @NonNull UserStartedCallback callback) { diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStopUserClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStopUserClient.java index 9d381459566a1..d9b0dfcc90bd9 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStopUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/FingerprintStopUserClient.java @@ -27,11 +27,13 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallback; import com.android.server.biometrics.sensors.StopUserClient; +import java.util.function.Supplier; + public class FingerprintStopUserClient extends StopUserClient { private static final String TAG = "FingerprintStopUserClient"; public FingerprintStopUserClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @Nullable IBinder token, int userId, + @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStoppedCallback callback) { super(context, lazyDaemon, token, userId, sensorId, callback); } diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/Sensor.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/Sensor.java index 59e4b582ca84e..e1e73a32661a3 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/Sensor.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/aidl/Sensor.java @@ -51,7 +51,6 @@ import com.android.server.biometrics.sensors.BaseClientMonitor; import com.android.server.biometrics.sensors.BiometricScheduler; import com.android.server.biometrics.sensors.EnumerateConsumer; import com.android.server.biometrics.sensors.ErrorConsumer; -import com.android.server.biometrics.sensors.HalClientMonitor; import com.android.server.biometrics.sensors.LockoutCache; import com.android.server.biometrics.sensors.LockoutConsumer; import com.android.server.biometrics.sensors.LockoutResetDispatcher; @@ -66,6 +65,7 @@ import com.android.server.biometrics.sensors.fingerprint.GestureAvailabilityDisp import java.util.ArrayList; import java.util.HashMap; import java.util.Map; +import java.util.function.Supplier; /** * Maintains the state of a single sensor within an instance of the @@ -87,7 +87,7 @@ class Sensor { @NonNull private final Map mAuthenticatorIds; @Nullable private Session mCurrentSession; - @NonNull private final HalClientMonitor.LazyDaemon mLazySession; + @NonNull private final Supplier mLazySession; static class Session { @NonNull private final String mTag; @@ -500,7 +500,7 @@ class Sensor { mLazySession = () -> mCurrentSession != null ? mCurrentSession.mSession : null; } - @NonNull HalClientMonitor.LazyDaemon getLazySession() { + @NonNull Supplier getLazySession() { return mLazySession; } diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/Fingerprint21.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/Fingerprint21.java index f160dfff52491..29d460fc12043 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/Fingerprint21.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/Fingerprint21.java @@ -67,7 +67,6 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.ClientMonitorCompositeCallback; import com.android.server.biometrics.sensors.EnumerateConsumer; import com.android.server.biometrics.sensors.ErrorConsumer; -import com.android.server.biometrics.sensors.HalClientMonitor; import com.android.server.biometrics.sensors.LockoutResetDispatcher; import com.android.server.biometrics.sensors.LockoutTracker; import com.android.server.biometrics.sensors.PerformanceTracker; @@ -90,6 +89,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.concurrent.atomic.AtomicLong; +import java.util.function.Supplier; /** * Supports a single instance of the {@link android.hardware.biometrics.fingerprint.V2_1} or @@ -111,7 +111,7 @@ public class Fingerprint21 implements IHwBinder.DeathRecipient, ServiceProvider private final LockoutResetDispatcher mLockoutResetDispatcher; private final LockoutFrameworkImpl mLockoutTracker; private final BiometricTaskStackListener mTaskStackListener; - private final HalClientMonitor.LazyDaemon mLazyDaemon; + private final Supplier mLazyDaemon; private final Map mAuthenticatorIds; @Nullable private IBiometricsFingerprint mDaemon; diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintAuthenticationClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintAuthenticationClient.java index 87d47c1cffc54..589bfcfba9928 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintAuthenticationClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintAuthenticationClient.java @@ -45,6 +45,7 @@ import com.android.server.biometrics.sensors.fingerprint.Udfps; import com.android.server.biometrics.sensors.fingerprint.UdfpsHelper; import java.util.ArrayList; +import java.util.function.Supplier; /** * Fingerprint-specific authentication client supporting the @@ -64,7 +65,7 @@ class FingerprintAuthenticationClient extends AuthenticationClient lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int targetUserId, long operationId, boolean restricted, @NonNull String owner, int cookie, boolean requireConfirmation, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintDetectClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintDetectClient.java index 9137212253e8b..88487462e51a1 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintDetectClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintDetectClient.java @@ -40,6 +40,7 @@ import com.android.server.biometrics.sensors.fingerprint.Udfps; import com.android.server.biometrics.sensors.fingerprint.UdfpsHelper; import java.util.ArrayList; +import java.util.function.Supplier; /** * Performs fingerprint detection without exposing any matching information (e.g. accept/reject @@ -55,7 +56,7 @@ class FingerprintDetectClient extends AcquisitionClient private boolean mIsPointerDown; public FingerprintDetectClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId, @Nullable IUdfpsOverlayController udfpsOverlayController, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintEnrollClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintEnrollClient.java index 82b046d0ffd2b..c69deac371d90 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintEnrollClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintEnrollClient.java @@ -41,6 +41,8 @@ import com.android.server.biometrics.sensors.SensorOverlays; import com.android.server.biometrics.sensors.fingerprint.Udfps; import com.android.server.biometrics.sensors.fingerprint.UdfpsHelper; +import java.util.function.Supplier; + /** * Fingerprint-specific enroll client supporting the * {@link android.hardware.biometrics.fingerprint.V2_1} and @@ -56,7 +58,7 @@ public class FingerprintEnrollClient extends EnrollClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, long requestId, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull byte[] hardwareAuthToken, @NonNull String owner, @NonNull BiometricUtils utils, int timeoutSec, int sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintGenerateChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintGenerateChallengeClient.java index db2f0455a4755..591f542396ef5 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintGenerateChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintGenerateChallengeClient.java @@ -26,6 +26,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.GenerateChallengeClient; +import java.util.function.Supplier; + /** * Fingerprint-specific generateChallenge/preEnroll client supporting the * {@link android.hardware.biometrics.fingerprint.V2_1} and @@ -37,7 +39,7 @@ public class FingerprintGenerateChallengeClient private static final String TAG = "FingerprintGenerateChallengeClient"; FingerprintGenerateChallengeClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, listener, userId, owner, sensorId); diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalCleanupClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalCleanupClient.java index a42a8ae24926d..403602b8839db 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalCleanupClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalCleanupClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.RemovalClient; import java.util.List; import java.util.Map; +import java.util.function.Supplier; /** * Fingerprint-specific internal cleanup client supporting the @@ -40,7 +41,7 @@ class FingerprintInternalCleanupClient extends InternalCleanupClient { FingerprintInternalCleanupClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, int userId, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, @NonNull List enrolledList, @NonNull BiometricUtils utils, @NonNull Map authenticatorIds) { @@ -50,7 +51,7 @@ class FingerprintInternalCleanupClient @Override protected InternalEnumerateClient getEnumerateClient( - Context context, LazyDaemon lazyDaemon, IBinder token, + Context context, Supplier lazyDaemon, IBinder token, int userId, String owner, List enrolledList, BiometricUtils utils, int sensorId) { return new FingerprintInternalEnumerateClient(context, lazyDaemon, token, userId, owner, @@ -59,7 +60,7 @@ class FingerprintInternalCleanupClient @Override protected RemovalClient getRemovalClient(Context context, - LazyDaemon lazyDaemon, IBinder token, + Supplier lazyDaemon, IBinder token, int biometricId, int userId, String owner, BiometricUtils utils, int sensorId, Map authenticatorIds) { // Internal remove does not need to send results to anyone. Cleanup (enumerate + remove) diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalEnumerateClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalEnumerateClient.java index 7117cf382a60b..def8ed0735f59 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalEnumerateClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintInternalEnumerateClient.java @@ -29,6 +29,7 @@ import com.android.server.biometrics.sensors.BiometricUtils; import com.android.server.biometrics.sensors.InternalEnumerateClient; import java.util.List; +import java.util.function.Supplier; /** * Fingerprint-specific internal enumerate client supporting the @@ -39,7 +40,7 @@ class FingerprintInternalEnumerateClient extends InternalEnumerateClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, @NonNull List enrolledList, @NonNull BiometricUtils utils, int sensorId) { super(context, lazyDaemon, token, userId, owner, enrolledList, utils, sensorId, diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRemovalClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRemovalClient.java index 2f360f31a325f..77c201c5ec97c 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRemovalClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRemovalClient.java @@ -30,6 +30,7 @@ import com.android.server.biometrics.sensors.ClientMonitorCallbackConverter; import com.android.server.biometrics.sensors.RemovalClient; import java.util.Map; +import java.util.function.Supplier; /** * Fingerprint-specific removal client supporting the @@ -42,7 +43,7 @@ class FingerprintRemovalClient extends RemovalClient lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener, int biometricId, int userId, @NonNull String owner, @NonNull BiometricUtils utils, int sensorId, @NonNull Map authenticatorIds) { diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRevokeChallengeClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRevokeChallengeClient.java index b6b29b32d909e..0180a466d7d60 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRevokeChallengeClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintRevokeChallengeClient.java @@ -25,6 +25,8 @@ import android.util.Slog; import com.android.server.biometrics.sensors.RevokeChallengeClient; +import java.util.function.Supplier; + /** * Fingerprint-specific revokeChallenge client supporting the * {@link android.hardware.biometrics.fingerprint.V2_1} and @@ -36,7 +38,7 @@ public class FingerprintRevokeChallengeClient private static final String TAG = "FingerprintRevokeChallengeClient"; FingerprintRevokeChallengeClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, int userId, @NonNull String owner, int sensorId) { super(context, lazyDaemon, token, userId, owner, sensorId); } diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintUpdateActiveUserClient.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintUpdateActiveUserClient.java index d317984c140d1..cb9c33eb9956a 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintUpdateActiveUserClient.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/hidl/FingerprintUpdateActiveUserClient.java @@ -49,7 +49,7 @@ public class FingerprintUpdateActiveUserClient extends HalClientMonitor lazyDaemon, int userId, + @NonNull Supplier lazyDaemon, int userId, @NonNull String owner, int sensorId, Supplier currentUserId, boolean hasEnrolledBiometrics, @NonNull Map authenticatorIds, boolean forceUpdateAuthenticatorId) { diff --git a/services/tests/servicestests/src/com/android/server/biometrics/sensors/AcquisitionClientTest.java b/services/tests/servicestests/src/com/android/server/biometrics/sensors/AcquisitionClientTest.java index fc55a9f4cf801..9bb722f8a853e 100644 --- a/services/tests/servicestests/src/com/android/server/biometrics/sensors/AcquisitionClientTest.java +++ b/services/tests/servicestests/src/com/android/server/biometrics/sensors/AcquisitionClientTest.java @@ -39,6 +39,8 @@ import org.junit.Test; import org.mockito.Mock; import org.mockito.MockitoAnnotations; +import java.util.function.Supplier; + @Presubmit @SmallTest public class AcquisitionClientTest { @@ -87,7 +89,7 @@ public class AcquisitionClientTest { boolean mHalOperationRunning; public TestAcquisitionClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter callback) { super(context, lazyDaemon, token, callback, 0 /* userId */, "Test", 0 /* cookie */, TEST_SENSOR_ID /* sensorId */, true /* shouldVibrate */, 0 /* statsModality */, diff --git a/services/tests/servicestests/src/com/android/server/biometrics/sensors/BiometricSchedulerTest.java b/services/tests/servicestests/src/com/android/server/biometrics/sensors/BiometricSchedulerTest.java index c99d656892f40..ecd9abcb80e64 100644 --- a/services/tests/servicestests/src/com/android/server/biometrics/sensors/BiometricSchedulerTest.java +++ b/services/tests/servicestests/src/com/android/server/biometrics/sensors/BiometricSchedulerTest.java @@ -61,6 +61,8 @@ import org.junit.runner.RunWith; import org.mockito.Mock; import org.mockito.MockitoAnnotations; +import java.util.function.Supplier; + @Presubmit @SmallTest @RunWith(AndroidTestingRunner.class) @@ -93,7 +95,7 @@ public class BiometricSchedulerTest { @Test public void testClientDuplicateFinish_ignoredBySchedulerAndDoesNotCrash() { - final HalClientMonitor.LazyDaemon nonNullDaemon = () -> mock(Object.class); + final Supplier nonNullDaemon = () -> mock(Object.class); final HalClientMonitor client1 = new TestHalClientMonitor(mContext, mToken, nonNullDaemon); @@ -184,7 +186,7 @@ public class BiometricSchedulerTest { @Test public void testCancelNotInvoked_whenOperationWaitingForCookie() { - final HalClientMonitor.LazyDaemon lazyDaemon1 = () -> mock(Object.class); + final Supplier lazyDaemon1 = () -> mock(Object.class); final TestAuthenticationClient client1 = new TestAuthenticationClient(mContext, lazyDaemon1, mToken, mock(ClientMonitorCallbackConverter.class)); final ClientMonitorCallback callback1 = mock(ClientMonitorCallback.class); @@ -296,7 +298,7 @@ public class BiometricSchedulerTest { @Test public void testCancelPendingAuth() throws RemoteException { - final HalClientMonitor.LazyDaemon lazyDaemon = () -> mock(Object.class); + final Supplier lazyDaemon = () -> mock(Object.class); final TestHalClientMonitor client1 = new TestHalClientMonitor(mContext, mToken, lazyDaemon); final ClientMonitorCallbackConverter callback = mock(ClientMonitorCallbackConverter.class); final TestAuthenticationClient client2 = new TestAuthenticationClient(mContext, lazyDaemon, @@ -360,7 +362,7 @@ public class BiometricSchedulerTest { private void testCancelsAuthDetectWhenRequestId(@Nullable Long requestId, long cancelRequestId, boolean started) { - final HalClientMonitor.LazyDaemon lazyDaemon = () -> mock(Object.class); + final Supplier lazyDaemon = () -> mock(Object.class); final ClientMonitorCallbackConverter callback = mock(ClientMonitorCallbackConverter.class); testCancelsWhenRequestId(requestId, cancelRequestId, started, new TestAuthenticationClient(mContext, lazyDaemon, mToken, callback)); @@ -383,7 +385,7 @@ public class BiometricSchedulerTest { private void testCancelsEnrollWhenRequestId(@Nullable Long requestId, long cancelRequestId, boolean started) { - final HalClientMonitor.LazyDaemon lazyDaemon = () -> mock(Object.class); + final Supplier lazyDaemon = () -> mock(Object.class); final ClientMonitorCallbackConverter callback = mock(ClientMonitorCallbackConverter.class); testCancelsWhenRequestId(requestId, cancelRequestId, started, new TestEnrollClient(mContext, lazyDaemon, mToken, callback)); @@ -441,7 +443,7 @@ public class BiometricSchedulerTest { public void testCancelsPending_whenAuthRequestIdsSet() { final long requestId1 = 10; final long requestId2 = 20; - final HalClientMonitor.LazyDaemon lazyDaemon = () -> mock(Object.class); + final Supplier lazyDaemon = () -> mock(Object.class); final ClientMonitorCallbackConverter callback = mock(ClientMonitorCallbackConverter.class); final TestAuthenticationClient client1 = new TestAuthenticationClient( mContext, lazyDaemon, mToken, callback); @@ -500,7 +502,7 @@ public class BiometricSchedulerTest { @Test public void testClientDestroyed_afterFinish() { - final HalClientMonitor.LazyDaemon nonNullDaemon = () -> mock(Object.class); + final Supplier nonNullDaemon = () -> mock(Object.class); final TestHalClientMonitor client = new TestHalClientMonitor(mContext, mToken, nonNullDaemon); mScheduler.scheduleClientMonitor(client); @@ -520,7 +522,7 @@ public class BiometricSchedulerTest { int mNumCancels = 0; public TestAuthenticationClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener) { super(context, lazyDaemon, token, listener, 0 /* targetUserId */, 0 /* operationId */, false /* restricted */, TAG, 1 /* cookie */, false /* requireConfirmation */, @@ -567,7 +569,7 @@ public class BiometricSchedulerTest { int mNumCancels = 0; TestEnrollClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @NonNull IBinder token, + @NonNull Supplier lazyDaemon, @NonNull IBinder token, @NonNull ClientMonitorCallbackConverter listener) { super(context, lazyDaemon, token, listener, 0 /* userId */, new byte[69], "test" /* owner */, mock(BiometricUtils.class), @@ -604,12 +606,12 @@ public class BiometricSchedulerTest { private boolean mDestroyed; TestHalClientMonitor(@NonNull Context context, @NonNull IBinder token, - @NonNull LazyDaemon lazyDaemon) { + @NonNull Supplier lazyDaemon) { this(context, token, lazyDaemon, 0 /* cookie */, BiometricsProto.CM_UPDATE_ACTIVE_USER); } TestHalClientMonitor(@NonNull Context context, @NonNull IBinder token, - @NonNull LazyDaemon lazyDaemon, int cookie, int protoEnum) { + @NonNull Supplier lazyDaemon, int cookie, int protoEnum) { super(context, lazyDaemon, token /* token */, null /* listener */, 0 /* userId */, TAG, cookie, TEST_SENSOR_ID, 0 /* statsModality */, 0 /* statsAction */, 0 /* statsClient */); diff --git a/services/tests/servicestests/src/com/android/server/biometrics/sensors/UserAwareBiometricSchedulerTest.java b/services/tests/servicestests/src/com/android/server/biometrics/sensors/UserAwareBiometricSchedulerTest.java index a11709aff87fe..25904e026b608 100644 --- a/services/tests/servicestests/src/com/android/server/biometrics/sensors/UserAwareBiometricSchedulerTest.java +++ b/services/tests/servicestests/src/com/android/server/biometrics/sensors/UserAwareBiometricSchedulerTest.java @@ -47,6 +47,8 @@ import org.junit.runner.RunWith; import org.mockito.Mock; import org.mockito.MockitoAnnotations; +import java.util.function.Supplier; + @Presubmit @RunWith(AndroidTestingRunner.class) @RunWithLooper @@ -223,7 +225,7 @@ public class UserAwareBiometricSchedulerTest { private static class TestStopUserClient extends StopUserClient { public TestStopUserClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @Nullable IBinder token, int userId, + @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStoppedCallback callback) { super(context, lazyDaemon, token, userId, sensorId, callback); } @@ -251,7 +253,7 @@ public class UserAwareBiometricSchedulerTest { ClientMonitorCallback mCallback; public TestStartUserClient(@NonNull Context context, - @NonNull LazyDaemon lazyDaemon, @Nullable IBinder token, int userId, + @NonNull Supplier lazyDaemon, @Nullable IBinder token, int userId, int sensorId, @NonNull UserStartedCallback callback, boolean shouldFinish) { super(context, lazyDaemon, token, userId, sensorId, callback); mShouldFinish = shouldFinish;