Bug 35566

Summary: WebKit needs ability to dispatch mousewheel events.
Product: WebKit Reporter: Andy Estes <aestes>
Component: DOMAssignee: Andy Estes <aestes>
Status: RESOLVED FIXED    
Severity: Enhancement CC: adele
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 29601    
Attachments:
Description Flags
Proposed patch
mjs: review+
Proposed patch (v2) mjs: review+

Andy Estes
Reported 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.
Attachments
Proposed patch (24.08 KB, patch)
2010-03-02 01:36 PST, Andy Estes 		mjs: review+
DetailsFormatted DiffDiff
Proposed patch (v2) (23.86 KB, patch)
2010-03-02 02:45 PST, Andy Estes 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andy Estes
Comment 1 2010-03-02 01:28:25 PST
<rdar://problem/7705815>
Andy Estes
Comment 2 2010-03-02 01:36:53 PST
Created attachment 49790 [details] Proposed patch
Maciej Stachowiak
Comment 3 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.
Andy Estes
Comment 4 2010-03-02 02:45:09 PST
Created attachment 49793 [details] Proposed patch (v2) Updated the patch to address Maciej's comments.
Maciej Stachowiak
Comment 5 2010-03-02 02:46:55 PST
Comment on attachment 49793 [details] Proposed patch (v2) r=me
Mark Rowe (bdash)
Comment 6 2010-03-02 15:34:14 PST
Landed in r55436 since the commit queue seems uninterested in committing changes.
Note You need to log in before you can comment on or make changes to this bug.