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

Commit 78ac57b0 authored by Ilya Matyukhin's avatar Ilya Matyukhin
Browse files

Tighten the scope of ACTION_UDFPS_ILLUMINATE 

This metric is not supposed to reflect the latency outside of the
illumination path, such as the time it takes the user to move their
finger to the trigger point on the sensor.

Bug: 221125251
Test: N/A
Change-Id: I47ea062a0527aed774ad5bd5e778a1def7f94601
parent b497365a

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

+1 −3
Original line number Diff line number Diff line
@@ -376,7 +376,6 @@ public class UdfpsController implements DozeReceiver { 
                boolean withinSensorArea =

                        isWithinSensorArea(udfpsView, event.getX(), event.getY(), fromUdfpsView);

                if (withinSensorArea) {

                    mLatencyTracker.onActionStart(LatencyTracker.ACTION_UDFPS_ILLUMINATE);

                    Trace.beginAsyncSection("UdfpsController.e2e.onPointerDown", 0);

                    Log.v(TAG, "onTouch | action down");

                    // The pointer that causes ACTION_DOWN is always at index 0.
@@ -792,6 +791,7 @@ public class UdfpsController implements DozeReceiver { 
                    + " current: " + mOverlay.getRequestId());

            return;

        }

        mLatencyTracker.onActionStart(LatencyTracker.ACTION_UDFPS_ILLUMINATE);



        if (!mOnFingerDown) {

            playStartHaptic();
@@ -806,11 +806,9 @@ public class UdfpsController implements DozeReceiver { 


        final UdfpsView view = mOverlay.getOverlayView();

        if (view != null) {

            Trace.beginAsyncSection("UdfpsController.e2e.startIllumination", 0);

            view.startIllumination(() -> {

                mFingerprintManager.onUiReady(requestId, mSensorProps.sensorId);

                mLatencyTracker.onActionEnd(LatencyTracker.ACTION_UDFPS_ILLUMINATE);

                Trace.endAsyncSection("UdfpsController.e2e.startIllumination", 0);

            });

        }