summaryrefslogtreecommitdiff
path: root/libs/gui/QueueBufferInputOutput.cpp
diff options
context:
space:
mode:
author Prabir Pradhan <prabirmsp@google.com> 2020-01-31 19:05:41 -0800
committer Prabir Pradhan <prabirmsp@google.com> 2020-01-31 19:05:41 -0800
commita940fc6f0c5c63c3edeee1ea1659c7fe2199cae3 (patch)
tree79c4acdc40332483d195af9dc7dfaa8b160f3d41 /libs/gui/QueueBufferInputOutput.cpp
parent07f36b934401b09a71938979fb841ffd374b7b6f (diff)
TouchInputMapper: Use default viewport for PointerController
There was a possibility of accidental mouse warping between multiple displays. InputReader has the responsibility of setting the DisplayViewport that the pointer is associated with, but framework sets a recommended "defaultPointerDisplayId". TouchInputMapper uses PointerController for touchpads (DEVICE_MODE_POINTER) and for when "showTouches" (touch spots) are enabled. In the latter case, TouchInputMapper was setting the pointer's DisplayViewport to the viewport that the mapper itself was associated with. When this viewport differs from the one recommended by framework, there is a possibility that the pointer is moved unexpectedly to the display associated with the mapper. The possibility of this happening depends upon the the order in which the various input mappers are configured. This CL removes the possibility of unexpected warping by making TouchInputMapper always use the viewport recommended by framework. Bug: 146385350 Test: atest inputflinger_tests Change-Id: Id73c59bce825855bc8c829c3a5a18c141593057e
Diffstat (limited to 'libs/gui/QueueBufferInputOutput.cpp')
0 files changed, 0 insertions, 0 deletions