commit 49f94b5919de0db0ada4e81f93219e8d4ffa95c6
author Byoungho Jung <bh.jung@samsung.corp-partner.google.com> Mon Oct 30 17:41:28 2023 +0900
committer Prabir Pradhan <prabirmsp@google.com> Wed Nov 08 20:29:42 2023 +0000
tree 70ed55b28c5002f1f181fb992e4b754f53be2461
parent 08daa25989491352b08ee10a33fae530b3bed12e

Modify getPosition of disabled PointerController

PointerChoreographer has to combine fake finger coordinates with
the position of MousePointerController to process some touchpad
events.
The legacy PointerController which is disabled should return
(0, 0) instead of (nan, nan) on getPosition, so that
GestureConverter can set x-y axes correctly.

Test: atest libinputservice_test
Bug: 293587049
Change-Id: I7630cd16defc9324970f0c29851c80f1b9943977

libs/input/PointerController.cpp

diff --git a/libs/input/PointerController.cpp b/libs/input/PointerController.cpp
index 972e168..2e7452d 100644
--- a/libs/input/PointerController.cpp
+++ b/libs/input/PointerController.cpp
@@ -175,8 +175,7 @@
 
 FloatPoint PointerController::getPosition() const {
     if (!mEnabled) {
-        return FloatPoint{AMOTION_EVENT_INVALID_CURSOR_POSITION,
-                          AMOTION_EVENT_INVALID_CURSOR_POSITION};
+        return FloatPoint{0, 0};
     }
 
     const int32_t displayId = mCursorController.getDisplayId();