Bug 89856

Summary: ASSERT(m_wheelEventHandlerCount > 0) can fire
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, jonlee, simon.fraser, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 66687    
Attachments:
Description Flags
Patch none

Adam Barth
Reported 2012-06-24 23:44:24 PDT
ASSERT(m_wheelEventHandlerCount > 0) can fire
Attachments
Patch (5.81 KB, patch)
2012-06-24 23:48 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-06-24 23:48:24 PDT
Created attachment 149249 [details] Patch
Eric Seidel (no email)
Comment 2 2012-06-25 08:17:04 PDT
Comment on attachment 149249 [details] Patch Wow. Why do we need to do this bookkeeping for wheel/mouse events?
Adam Barth
Comment 3 2012-06-25 09:17:01 PDT
> Wow. Why do we need to do this bookkeeping for wheel/mouse events? I'm not sure. I'll investigate before landing.
Adam Barth
Comment 4 2012-06-25 09:46:30 PDT
Comment on attachment 149249 [details] Patch It's used by the fast scrolling code, presumably to know whether the fast scrolling code can eat these events and scroll fast.
WebKit Review Bot
Comment 5 2012-06-25 10:39:32 PDT
Comment on attachment 149249 [details] Patch Clearing flags on attachment: 149249 Committed r121166: <http://trac.webkit.org/changeset/121166>
WebKit Review Bot
Comment 6 2012-06-25 10:39:46 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.