Bug 35566 - WebKit needs ability to dispatch mousewheel events.
Summary: WebKit needs ability to dispatch mousewheel events.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Andy Estes
URL:
Keywords: InRadar
Depends on:
Blocks: 29601
  Show dependency treegraph
 
Reported: 2010-03-02 00:39 PST by Andy Estes
Modified: 2010-03-02 15:34 PST (History)
1 user (show)

See Also:


Attachments
Proposed patch (24.08 KB, patch)
2010-03-02 01:36 PST, Andy Estes
mjs: review+
Details | Formatted Diff | Diff
Proposed patch (v2) (23.86 KB, patch)
2010-03-02 02:45 PST, Andy Estes
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Estes 2010-03-02 00:39:39 PST
WebKit needs the ability to dispatch mouse wheel events from JavaScript in order to test WebKit scrolling behavior programatically.  For instance, https://bugs.webkit.org/show_bug.cgi?id=29601 cannot be properly tested without such a capability.
Comment 1 Andy Estes 2010-03-02 01:28:25 PST
<rdar://problem/7705815>
Comment 2 Andy Estes 2010-03-02 01:36:53 PST
Created attachment 49790 [details]
Proposed patch
Comment 3 Maciej Stachowiak 2010-03-02 02:17:11 PST
Comment on attachment 49790 [details]
Proposed patch

Suggestions from IRC: 
- instead of the long comments explaining the difference between m_wheelDeltaX/Y and m_deltaX/Y, I would suggest giving them names that make the difference more clear
- m_wheelDeltaX/Y are exposed by that name in the API, perhaps the others could be m_rawDeltaX/Y or m_pixelDeltaX/Y

r=me with that change.
Comment 4 Andy Estes 2010-03-02 02:45:09 PST
Created attachment 49793 [details]
Proposed patch (v2)

Updated the patch to address Maciej's comments.
Comment 5 Maciej Stachowiak 2010-03-02 02:46:55 PST
Comment on attachment 49793 [details]
Proposed patch (v2)

r=me
Comment 6 Mark Rowe (bdash) 2010-03-02 15:34:14 PST
Landed in r55436 since the commit queue seems uninterested in committing changes.