New issue
Advanced search Search tips

Issue 678291 link

Starred by 1 user
Status: Assigned
Owner:
eirage@chromium.org
Cc:
wjmaclean@chromium.org
dtapu...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Inconsistent coordinate/delta conversion when sending wheel events to plugins

Project Member Reported by majidvp@chromium.org, Jan 4 2017 
WebMouseWheelEventBuilder is used to convert platform wheel event to WebInputEvents that can then sent to a plugin.

In this process the coordinates  and converted from root frame 
space to plugin space (see [1]) using |layoutItem.absoluteToLocal|. This conversion may include a scale factor but this scaling is not applied to 
movement{X,Y} and delta{X,Y} which is inconsistent and incorrect.


[1] https://codesearch.chromium.org/chromium/src/third_party/WebKit/Source/web/WebInputEventConversion.cpp?q=WebMouseWheelEventBuilder&sq=package:chromium&dr=CSs&l=495

Comment 1 by dtapu...@chromium.org, Sep 8 2017

Components: Blink>Input
Labels: Hotlist-Input-Dev
Specifically 

Pepper conversion uses the raw values not the transformed values:

https://codesearch.chromium.org/chromium/src/content/renderer/pepper/event_conversion.cc?type=cs&l=235
Project Member

Comment 2 by sheriffbot@chromium.org, Sep 10

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by nzolghadr@chromium.org, Sep 13

Status: Available (was: Untriaged)
Still an issue in 2018 Q3. But there doesn't seem to be many starts on this issue. Keeping it as P3 for now.

Comment 4 by mustaq@chromium.org, Nov 19

Owner: eirage@chromium.org
Status: Assigned (was: Available)
Assigning to eirage@ but I agree it's not urgent.
►

Sign in to add a comment