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 tree / graph
 
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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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.