Commit 10af5a87 authored by Lan Wei's avatar Lan Wei Committed by Commit Bot

Set touch event type correctly when the touch point pauses at the tick

Because we can use pause action for touch points that do not move, but
Devtool will generate a touch end event if we do not pass the paused
touch point in the touch point list.
https://cs.chromium.org/chromium/src/content/browser/devtools/protocol/
input_handler.cc?type=cs&q=InputHandler::DispatchWebTouchEvent&g=0&l=825

Therefor, for the paused touch points that are not released, we should
set type of touch move.

Bug: 1020674
Change-Id: I01358b3534a8dde6b2651495034b9ad31a5f13ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1933206Reviewed-by: default avatarJohn Chen <johnchen@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#718912}
parent 3af9b87e
...@@ -1503,6 +1503,8 @@ Status ExecutePerformActions(Session* session, ...@@ -1503,6 +1503,8 @@ Status ExecutePerformActions(Session* session,
action_input_states[j]->SetInteger("pressed", 0); action_input_states[j]->SetInteger("pressed", 0);
} }
if (has_touch_start[id]) { if (has_touch_start[id]) {
if (event.type == kPause)
event.type = kTouchMove;
event.id = dispatch_touch_events.size(); event.id = dispatch_touch_events.size();
dispatch_touch_events.push_back(event); dispatch_touch_events.push_back(event);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment