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

Commit 8e36f822 authored by Prabir Pradhan's avatar Prabir Pradhan Committed by Automerger Merge Worker
Browse files

Get mouse cursor position from PointerController instead of WM am: 692bbdb3 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21566609



Change-Id: Ifa4a7e27faa69241c3e4f82229775a119ca2f3c7
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 355324c6 692bbdb3

libs/input/PointerController.cpp

+27 −12
Original line number Diff line number Diff line
@@ -262,6 +262,13 @@ void PointerController::reloadPointerResources() { 
}



void PointerController::setDisplayViewport(const DisplayViewport& viewport) {

    struct PointerDisplayChangeArgs {

        int32_t displayId;

        float x, y;

    };

    std::optional<PointerDisplayChangeArgs> pointerDisplayChanged;



    { // acquire lock

        std::scoped_lock lock(getLock());



        bool getAdditionalMouseResources = false;
@@ -273,8 +280,16 @@ void PointerController::setDisplayViewport(const DisplayViewport& viewport) { 
        if (viewport.displayId != mLocked.pointerDisplayId) {

            float xPos, yPos;

            mCursorController.getPosition(&xPos, &yPos);

        mContext.getPolicy()->onPointerDisplayIdChanged(viewport.displayId, xPos, yPos);

            mLocked.pointerDisplayId = viewport.displayId;

            pointerDisplayChanged = {viewport.displayId, xPos, yPos};

        }

    } // release lock



    if (pointerDisplayChanged) {

        // Notify the policy without holding the pointer controller lock.

        mContext.getPolicy()->onPointerDisplayIdChanged(pointerDisplayChanged->displayId,

                                                        pointerDisplayChanged->x,

                                                        pointerDisplayChanged->y);

    }

}

services/core/java/com/android/server/input/InputManagerInternal.java

+5 −1
Original line number Diff line number Diff line
@@ -96,7 +96,11 @@ public abstract class InputManagerInternal { 
     */

    public abstract int getVirtualMousePointerDisplayId();



    /** Gets the current position of the mouse cursor. */

    /**

     * Gets the current position of the mouse cursor.

     *

     * Returns NaN-s as the coordinates if the cursor is not available.

     */

    public abstract PointF getCursorPosition();



    /**

services/core/java/com/android/server/input/InputManagerService.java

+5 −4
Original line number Diff line number Diff line
@@ -2863,9 +2863,6 @@ public class InputManagerService extends IInputManager.Stub 


        int getPointerDisplayId();



        /** Gets the x and y coordinates of the cursor's current position. */

        PointF getCursorPosition();



        /**

         * Notifies window manager that a {@link android.view.MotionEvent#ACTION_DOWN} pointer event

         * occurred on a window that did not have focus.
@@ -3189,7 +3186,11 @@ public class InputManagerService extends IInputManager.Stub 


        @Override

        public PointF getCursorPosition() {

            return mWindowManagerCallbacks.getCursorPosition();

            final float[] p = mNative.getMouseCursorPosition();

            if (p == null || p.length != 2) {

                throw new IllegalStateException("Failed to get mouse cursor position");

            }

            return new PointF(p[0], p[1]);

        }



        @Override

services/core/java/com/android/server/input/NativeInputManagerService.java

+13 −0
Original line number Diff line number Diff line
@@ -224,6 +224,16 @@ interface NativeInputManagerService { 
    /** Set whether stylus button reporting through motion events should be enabled. */

    void setStylusButtonMotionEventsEnabled(boolean enabled);



    /**

     * Get the current position of the mouse cursor.

     *

     * If the mouse cursor is not currently shown, the coordinate values will be NaN-s.

     *

     * NOTE: This will grab the PointerController's lock, so we must be careful about calling this

     * from the InputReader or Display threads, which may result in a deadlock.

     */

    float[] getMouseCursorPosition();



    /** The native implementation of InputManagerService methods. */

    class NativeImpl implements NativeInputManagerService {

        /** Pointer to native input manager service object, used by native code. */
@@ -465,5 +475,8 @@ interface NativeInputManagerService { 


        @Override

        public native void setStylusButtonMotionEventsEnabled(boolean enabled);



        @Override

        public native float[] getMouseCursorPosition();

    }

}

services/core/java/com/android/server/wm/InputManagerCallback.java

+0 −6
Original line number Diff line number Diff line
@@ -25,7 +25,6 @@ import static com.android.server.wm.WindowManagerService.H.ON_POINTER_DOWN_OUTSI 


import android.annotation.NonNull;

import android.annotation.Nullable;

import android.graphics.PointF;

import android.os.Debug;

import android.os.IBinder;

import android.util.Slog;
@@ -220,11 +219,6 @@ final class InputManagerCallback implements InputManagerService.WindowManagerCal 
        }

    }



    @Override

    public PointF getCursorPosition() {

        return mService.getLatestMousePosition();

    }



    @Override

    public void onPointerDownOutsideFocus(IBinder touchedToken) {

        mService.mH.obtainMessage(ON_POINTER_DOWN_OUTSIDE_FOCUS, touchedToken).sendToTarget();