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

Commit 8ce6b9fb authored by Kevin Chyn's avatar Kevin Chyn Committed by Android (Google) Code Review
Browse files

Merge changes I96fe9587,Ie51b121b into sc-dev 

* changes:
  Request cancelAllSensors when STATE_AUTH_CALLED
  Add BaseClientMonitor#interruptsPrecedingClients
parents d438be4b 0c1be610

services/core/java/com/android/server/biometrics/AuthSession.java

+2 −1
Original line number Diff line number Diff line
@@ -686,7 +686,8 @@ public final class AuthSession implements IBinder.DeathRecipient { 
     * @return true if this AuthSession is finished, e.g. should be set to null

     */

    boolean onCancelAuthSession(boolean force) {

        final boolean authStarted = mState == STATE_AUTH_STARTED

        final boolean authStarted = mState == STATE_AUTH_CALLED

                || mState == STATE_AUTH_STARTED

                || mState == STATE_AUTH_STARTED_UI_SHOWING;



        if (authStarted && !force) {

services/core/java/com/android/server/biometrics/sensors/AuthenticationClient.java

+5 −0
Original line number Diff line number Diff line
@@ -311,4 +311,9 @@ public abstract class AuthenticationClient<T> extends AcquisitionClient<T> 
    public int getProtoEnum() {

        return BiometricsProto.CM_AUTHENTICATE;

    }



    @Override

    public boolean interruptsPrecedingClients() {

        return true;

    }

}

services/core/java/com/android/server/biometrics/sensors/BaseClientMonitor.java

+8 −2
Original line number Diff line number Diff line
@@ -82,11 +82,17 @@ public abstract class BaseClientMonitor extends LoggableMonitor 
    @NonNull protected Callback mCallback;



    /**

     * Returns a ClientMonitorEnum constant defined in biometrics.proto

     * @return

     * @return A ClientMonitorEnum constant defined in biometrics.proto

     */

    public abstract int getProtoEnum();



    /**

     * @return True if the ClientMonitor should cancel any current and pending interruptable clients

     */

    public boolean interruptsPrecedingClients() {

        return false;

    }



    /**

     * @param context    system_server context

     * @param token      a unique token for the client

services/core/java/com/android/server/biometrics/sensors/BiometricScheduler.java

+16 −10
Original line number Diff line number Diff line
@@ -475,8 +475,10 @@ public class BiometricScheduler { 
     */

    public void scheduleClientMonitor(@NonNull BaseClientMonitor clientMonitor,

            @Nullable BaseClientMonitor.Callback clientCallback) {

        // Mark any interruptable pending clients as canceling. Once they reach the head of the

        // queue, the scheduler will send ERROR_CANCELED and skip the operation.

        // If the incoming operation should interrupt preceding clients, mark any interruptable

        // pending clients as canceling. Once they reach the head of the queue, the scheduler will

        // send ERROR_CANCELED and skip the operation.

        if (clientMonitor.interruptsPrecedingClients()) {

            for (Operation operation : mPendingOperations) {

                if (operation.mClientMonitor instanceof Interruptable

                        && operation.mState != Operation.STATE_WAITING_IN_QUEUE_CANCELING) {
@@ -485,13 +487,17 @@ public class BiometricScheduler { 
                    operation.mState = Operation.STATE_WAITING_IN_QUEUE_CANCELING;

                }

            }

        }



        mPendingOperations.add(new Operation(clientMonitor, clientCallback));

        Slog.d(getTag(), "[Added] " + clientMonitor

                + ", new queue size: " + mPendingOperations.size());



        // If the current operation is cancellable, start the cancellation process.

        if (mCurrentOperation != null && mCurrentOperation.mClientMonitor instanceof Interruptable

        // If the new operation should interrupt preceding clients, and if the current operation is

        // cancellable, start the cancellation process.

        if (clientMonitor.interruptsPrecedingClients()

                && mCurrentOperation != null

                && mCurrentOperation.mClientMonitor instanceof Interruptable

                && mCurrentOperation.mState == Operation.STATE_STARTED) {

            Slog.d(getTag(), "[Cancelling Interruptable]: " + mCurrentOperation);

            cancelInternal(mCurrentOperation);

services/core/java/com/android/server/biometrics/sensors/EnrollClient.java

+5 −0
Original line number Diff line number Diff line
@@ -113,4 +113,9 @@ public abstract class EnrollClient<T> extends AcquisitionClient<T> { 
    public int getProtoEnum() {

        return BiometricsProto.CM_ENROLL;

    }



    @Override

    public boolean interruptsPrecedingClients() {

        return true;

    }

}