Summary: | [chromium] WebInputEventBuilders should not reverse page scale | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alexandre Elias <aelias> | ||||
Component: | New Bugs | Assignee: | Alexandre Elias <aelias> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aelias, jamesr, qinmin, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Alexandre Elias
2013-02-14 23:41:58 PST
Created attachment 188493 [details]
Patch
The lack of idempotence is unfortunate, but this is what turns out to actually work. Comment on attachment 188493 [details]
Patch
Hmmmm....odd, but them's the breaks I guess. Longer-term, do you think we should move the page scale adjustements for plugins somewhere else in the stack?
It's not clear to me what would be cleaner long-term... plugins are reusing WebInputEvents just because it lets them get the information back across to the Chromium side. But even in the absence of page scale, the fields in them don't have the same meaning -- the scroll origin is the origin of the plugin, not the window scroll position. Comment on attachment 188493 [details] Patch Clearing flags on attachment: 188493 Committed r143032: <http://trac.webkit.org/changeset/143032> All reviewed patches have been landed. Closing bug. |