New issue
Advanced search Search tips

Issue 713745 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: 5
NextAction: 2017-04-28
OS: All
Pri: 3
Type: Bug

Blocking:
issue 367113



Sign in to add a comment

Make sure PointerEvents' isPrimary set correctly for stylus

Project Member Reported by lanwei@chromium.org, Apr 20 2017

Issue description

Make sure PointerEvents's isPrimary set correctly for stylus

When a pen enters and leaves a tablet's proximity and enters again, the PointerEvents' isPrimary is always true.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 28 2017

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

commit c007949541f4d2238c9653a4e3b8dbc0987bc752
Author: lanwei <lanwei@chromium.org>
Date: Fri Apr 28 15:06:21 2017

Make sure PointerEvents's isPrimary set correctly for stylus

When a pen enters and leaves a tablet's proximity and enters again, the
PointerEvents' isPrimary should always be true.

Testing page: https://patrickhlauke.github.io/touch/tracker/multi-touch-tracker-pointer-hud.html

Webplatform test pointerevent_pointerleave_pen-manual.html does not work
right now because of no support of PointerLeave events when pen leaves
the range of the tablet in PointerActionSequence API. I will add it soon.

BUG= 713745 

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

[modify] https://crrev.com/c007949541f4d2238c9653a4e3b8dbc0987bc752/third_party/WebKit/LayoutTests/external/wpt/pointerevents/pointerevent_pointerleave_pen-manual.html
[modify] https://crrev.com/c007949541f4d2238c9653a4e3b8dbc0987bc752/third_party/WebKit/LayoutTests/external/wpt_automation/pointerevents/pointerevent_pointerleave_pen-manual-automation.js
[modify] https://crrev.com/c007949541f4d2238c9653a4e3b8dbc0987bc752/third_party/WebKit/Source/core/input/PointerEventManager.cpp

Comment 2 by lanwei@chromium.org, Apr 28 2017

Summary: Make sure PointerEvents' isPrimary set correctly for stylus (was: Make sure PointerEvents's isPrimary set correctly for stylus)

Comment 3 by lanwei@chromium.org, Apr 28 2017

Blocking: 367113
Status: Fixed (was: Assigned)

Sign in to add a comment