Bug 42987 – [chromium] Absolute Mouse coordinates getting lost in translation

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Gene Gutnik]

In chromium absolute mouse coordinates getting lost and only adjusted (multiplied by zoom factor) saved. In caused controls and plugins on the zoomed in/out pages to respond incorrectly to mouse events.

[Gene Gutnik]

Created attachment 62601 [details]
Added absolute (not multiplied by zoom level) mouse coordinates to WebMouseEvent

Added absolute (not multiplied by zoom level) mouse coordinates to WebMouseEvent

[Nate Chapin]

Comment on attachment 62601 [details]
Added absolute (not multiplied by zoom level) mouse coordinates to WebMouseEvent

The code looks fine to me, but you need to update the auto-generated ChangeLog entry with a real bug URL and description (the first line with OOPS! will be auto-corrected by the commit-queue, but the second won't).

[Gene Gutnik]

Created attachment 62605 [details]
Added bug URL and description to the ChangeLog

Thanks Nate,
I've added bug URL and description to the ChangeLog

[Nate Chapin]

Comment on attachment 62605 [details]
Added bug URL and description to the ChangeLog

Excellent, thanks!

I'm going to wait to actually send this to commit-queue until Darin has had a chance to look at it.

[Darin Fisher (:fishd, Google)]

Comment on attachment 62605 [details]
Added bug URL and description to the ChangeLog

WebKit/chromium/public/WebInputEvent.h:244
 +      int absoluteX;
I think this really deserves some comments to explain the meaning
of all of these coordinates.  It was possible to guess before, but
now it could be pretty confusing to developers.

Also, did you confirm that NPAPI plugins still require zoomed
coordinates?  I'm concerned that we are adding absoluteX,Y when
in fact we should just be changing the meaning of windowX,Y.

[Gene Gutnik]

Created attachment 62875 [details]
Fix mouse coordinate conversion. WindowX,Y is an absolute mouse coordinates within browser window, x,y - are coordinates relative to the origin of the widget.

Fix mouse coordinate conversion. WindowX,Y is an absolute mouse coordinates within browser window, x,y - are coordinates relative to the origin of the widget.

[Darin Fisher (:fishd, Google)]

Landed as http://trac.webkit.org/changeset/64243

This fixes bug 26424. https://bugs.webkit.org/show_bug.cgi?id=26424