|
|
Descriptionozone: evdev: Fix ghost touches aligned with first touch
The ABS_X and ABS_Y axes are not part of the MT protocol and should
therefore not be used to update MT slot positions.
The symptom of this was secondary touches warping to the position of the
first touch in the X and/or Y axis, since ABS_X/Y always corresponds to
the first touch.
BUG=407386
TEST=chrome on pixel with --ash-touch-hud
NOTRY=true
Committed: https://crrev.com/86c315c379bfeefe3dd833cee678a41ac6884a74
Cr-Commit-Position: refs/heads/master@{#292263}
Patch Set 1 #Patch Set 2 : add unit test #
Messages
Total messages: 31 (0 generated)
spang@chromium.org changed reviewers: + alexst@chromium.org, miletus@chromium.org
miletus@chromium.org changed reviewers: + sadrul@chromium.org
Does anyone care about ABS_X/Y? explicitly? If not, lgtm.
LGTM A test in touch_event_converter_evdev_unittest.cc?
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/1
The CQ bit was unchecked by spang@chromium.org
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Failed to commit the patch.
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Failed to commit the patch.
The CQ bit was checked by alexst@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Failed to commit the patch.
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
The CQ bit was unchecked by spang@chromium.org
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
The CQ bit was unchecked by commit-bot@chromium.org
Failed to commit the patch.
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/spang@chromium.org/506303002/20001
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as f59f314d1929edc62c5873858fa96308f75738d5
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/86c315c379bfeefe3dd833cee678a41ac6884a74 Cr-Commit-Position: refs/heads/master@{#292263} |