New issue
Advanced search Search tips

Issue 621367 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Touchscreen emulation does not emulate mousover events properly

Reported by teo8...@gmail.com, Jun 19 2016

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36

Steps to reproduce the problem:
1. On Desktop Chrome, open the developer tools and activate mobile emulation
2. visit http://output.jsbin.com/kikivel
3. "touch" (i.e. click) any of the two squares
4. Repeat the same with a real mobile device

What is the expected behavior?
The results should be the same

What went wrong?
On both the real mobile device and the emulated one, when you touch the square, a mouseover event is fired immediately followed by a click event.

However, while on the real mobile device the "buttons" property of the event has value 0, on Chrome's DevTools mobile emulator, the "buttons" property has value 1.

Did this work before? N/A 

Chrome version: 51.0.2704.103  Channel: stable
OS Version: 
Flash Version: Shockwave Flash 22.0 r0
 
Owner: dgozman@chromium.org
Status: Assigned (was: Unconfirmed)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 22 2016

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

commit 3af175cfb5381e32910b08ae3869eb5a5e19c0db
Author: dgozman <dgozman@chromium.org>
Date: Wed Jun 22 21:04:58 2016

Touch emulator: clear mouse buttons modifiers on emulated touch events.

Real touch events don't have mouse buttons modifiers which affect
buttons property of synthesized from tap mouse events.

BUG= 621367 

Review-Url: https://codereview.chromium.org/2084043003
Cr-Commit-Position: refs/heads/master@{#401399}

[modify] https://crrev.com/3af175cfb5381e32910b08ae3869eb5a5e19c0db/content/browser/renderer_host/input/touch_emulator.cc
[modify] https://crrev.com/3af175cfb5381e32910b08ae3869eb5a5e19c0db/content/browser/renderer_host/input/touch_emulator_unittest.cc

Status: Fixed (was: Assigned)

Sign in to add a comment