New issue
Advanced search Search tips

Issue 856638 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: 0
NextAction: 2018-07-06
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

SyntheticPointerAction does not handle multiple touch pointers correctly

Project Member Reported by lanwei@chromium.org, Jun 26 2018

Issue description

The current SyntheticPointerAction does not handle multiple touch pointers correctly. We should support different actions from multiple touch pointers and no matter which pointer release first.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0e114bd492826a638c43acd6d9e4c3d050d0671e

commit 0e114bd492826a638c43acd6d9e4c3d050d0671e
Author: lanwei <lanwei@chromium.org>
Date: Wed Jun 27 17:42:29 2018

Support multiple touch pointers

There was an issue when we handled multiple touch pointer actions.
SyntheticPointerAction should handle different actions from multiple
touch pointers.

Bug:  856638 
Change-Id: I771181bcfe452631074c95874fd5c1a7807d3e74
Reviewed-on: https://chromium-review.googlesource.com/1113898
Commit-Queue: Lan Wei <lanwei@chromium.org>
Reviewed-by: Ella Ge <eirage@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570829}
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/content/browser/renderer_host/input/synthetic_gesture_controller_unittest.cc
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/content/browser/renderer_host/input/synthetic_pointer_action.cc
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/content/browser/renderer_host/input/synthetic_pointer_action_unittest.cc
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/content/browser/renderer_host/ui_events_helper.cc
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/content/common/input/synthetic_web_input_event_builders.cc
[modify] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/0e114bd492826a638c43acd6d9e4c3d050d0671e/third_party/WebKit/LayoutTests/fast/events/pointerevents/multi-touch-events.html

Comment 2 by lanwei@chromium.org, Jun 28 2018

Status: Fixed (was: Started)
The NextAction date has arrived: 2018-07-06

Sign in to add a comment