Bug 89856 - ASSERT(m_wheelEventHandlerCount > 0) can fire
Summary: ASSERT(m_wheelEventHandlerCount > 0) can fire
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks: 66687
  Show dependency tree / graph
 
Reported: 2012-06-24 23:44 PDT by Adam Barth
Modified: 2012-06-25 10:39 PDT (History)
4 users (show)

See Also:

Attachments
Patch (5.81 KB, patch)
2012-06-24 23:48 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
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 Adam Barth 2012-06-24 23:44:24 PDT 
ASSERT(m_wheelEventHandlerCount > 0) can fire
Comment 1 Adam Barth 2012-06-24 23:48:24 PDT 
Created attachment 149249 [details]
Patch
Comment 2 Eric Seidel (no email) 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?
Comment 3 Adam Barth 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.
Comment 4 Adam Barth 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.
Comment 5 WebKit Review Bot 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>
Comment 6 WebKit Review Bot 2012-06-25 10:39:46 PDT 
All reviewed patches have been landed.  Closing bug.