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

Commit 79ea4826 authored by Jackal Guo's avatar Jackal Guo
Browse files

Using transformed event for standard gesture detector 

TouchExplorer uses raw event in both a11y and standard gestures
detection, but transformed event in following movement. This
inconsistency causes significant difference of distance between
their locations when screen is magnified and long press would fail.
Passing transformed events for standard gesture detector instead.

Bug: 67378068
Test: manually test if double tap and hold works while Talkback
and Magnification are both enabled.
Test: manually test if Talkback and Magnification work as expected.
Test: AccessibilityGestureDetectorTest

Change-Id: I0e1dc59e25bc36f1f99ba6ecdb05c9c488b6f47e
parent 740be2ef

services/accessibility/java/com/android/server/accessibility/AccessibilityGestureDetector.java

+16 −13
Original line number Diff line number Diff line
@@ -235,14 +235,15 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
     * callback on mListener is called, and the return value of the callback is

     * passed to the caller.

     *

     * @param event The raw motion event.  It's important that this be the raw

     * @param event The transformed motion event to be handled.

     * @param rawEvent The raw motion event.  It's important that this be the raw

     * event, before any transformations have been applied, so that measurements

     * can be made in physical units.

     * @param policyFlags Policy flags for the event.

     *

     * @return true if the event is consumed, else false

     */

    public boolean onMotionEvent(MotionEvent event, int policyFlags) {

    public boolean onMotionEvent(MotionEvent event, MotionEvent rawEvent, int policyFlags) {



        // Construct GestureDetector double-tap detector on demand, so that testable sub-class

        // can use mock GestureDetector.
@@ -255,12 +256,14 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
            mGestureDetector.setOnDoubleTapListener(this);

        }



        final float x = event.getX();

        final float y = event.getY();

        final long time = event.getEventTime();

        // The accessibility gesture detector is interested in the movements in physical space,

        // so it uses the rawEvent to ignore magnification and other transformations.

        final float x = rawEvent.getX();

        final float y = rawEvent.getY();

        final long time = rawEvent.getEventTime();



        mPolicyFlags = policyFlags;

        switch (event.getActionMasked()) {

        switch (rawEvent.getActionMasked()) {

            case MotionEvent.ACTION_DOWN:

                mDoubleTapDetected = false;

                mSecondFingerDoubleTap = false;
@@ -311,7 +314,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
                        // timeout, cancel gesture detection.

                        if (timeDelta > threshold) {

                            cancelGesture();

                            return mListener.onGestureCancelled(event, policyFlags);

                            return mListener.onGestureCancelled(rawEvent, policyFlags);

                        }

                    }
@@ -327,7 +330,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 


            case MotionEvent.ACTION_UP:

                if (mDoubleTapDetected) {

                    return finishDoubleTap(event, policyFlags);

                    return finishDoubleTap(rawEvent, policyFlags);

                }

                if (mGestureStarted) {

                    final float dX = Math.abs(x - mPreviousGestureX);
@@ -335,7 +338,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
                    if (dX >= mMinPixelsBetweenSamplesX || dY >= mMinPixelsBetweenSamplesY) {

                        mStrokeBuffer.add(new GesturePoint(x, y, time));

                    }

                    return recognizeGesture(event, policyFlags);

                    return recognizeGesture(rawEvent, policyFlags);

                }

                break;
@@ -344,7 +347,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
                // recognizing a gesture.

                cancelGesture();



                if (event.getPointerCount() == 2) {

                if (rawEvent.getPointerCount() == 2) {

                    // If this was the second finger, attempt to recognize double

                    // taps on it.

                    mSecondFingerDoubleTap = true;
@@ -360,7 +363,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
                // If we're detecting taps on the second finger, see if we

                // should finish the double tap.

                if (mSecondFingerDoubleTap && mDoubleTapDetected) {

                    return finishDoubleTap(event, policyFlags);

                    return finishDoubleTap(rawEvent, policyFlags);

                }

                break;
@@ -372,7 +375,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
        // If we're detecting taps on the second finger, map events from the

        // finger to the first finger.

        if (mSecondFingerDoubleTap) {

            MotionEvent newEvent = mapSecondPointerToFirstPointer(event);

            MotionEvent newEvent = mapSecondPointerToFirstPointer(rawEvent);

            if (newEvent == null) {

                return false;

            }
@@ -385,7 +388,7 @@ class AccessibilityGestureDetector extends GestureDetector.SimpleOnGestureListen 
            return false;

        }



        // Pass the event on to the standard gesture detector.

        // Pass the transformed event on to the standard gesture detector.

        return mGestureDetector.onTouchEvent(event);

    }

services/accessibility/java/com/android/server/accessibility/TouchExplorer.java

+1 −4
Original line number Diff line number Diff line
@@ -268,10 +268,7 @@ class TouchExplorer extends BaseEventStreamTransformation 


        mReceivedPointerTracker.onMotionEvent(rawEvent);



        // The motion detector is interested in the movements in physical space,

        // so it uses the rawEvent to ignore magnification and other

        // transformations.

        if (mGestureDetector.onMotionEvent(rawEvent, policyFlags)) {

        if (mGestureDetector.onMotionEvent(event, rawEvent, policyFlags)) {

            // Event was handled by the gesture detector.

            return;

        }

services/tests/servicestests/src/com/android/server/accessibility/AccessibilityGestureDetectorTest.java

+1 −1
Original line number Diff line number Diff line
@@ -195,7 +195,7 @@ public class AccessibilityGestureDetectorTest { 
                    point.x, point.y, 0);



            // Send event.

            mDetector.onMotionEvent(event, policyFlags);

            mDetector.onMotionEvent(event, event, policyFlags);

            eventTimeMs += PATH_STEP_MILLISEC;

        }