Bug 120264 - No need for notifyChromeClientWheelEventHandlerCountChanged in Frame
Summary: No need for notifyChromeClientWheelEventHandlerCountChanged in Frame
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-25 00:35 PDT by Darin Adler
Modified: 2013-08-25 02:53 PDT (History)
4 users (show)

See Also:

Attachments
Patch (12.62 KB, patch)
2013-08-25 00:54 PDT, Darin Adler 		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 Darin Adler 2013-08-25 00:35:57 PDT 
No need for notifyChromeClientWheelEventHandlerCountChanged in Frame
Comment 1 Darin Adler 2013-08-25 00:54:10 PDT 
Created attachment 209575 [details]
Patch
Comment 2 Andreas Kling 2013-08-25 02:27:48 PDT 
Comment on attachment 209575 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=209575&action=review

This looks great.

> Source/WebCore/dom/Document.cpp:2026
> +        page()->chrome().client().needTouchEvents(true);

Who comes up with these names? :|
Comment 3 WebKit Commit Bot 2013-08-25 02:53:06 PDT 
Comment on attachment 209575 [details]
Patch

Clearing flags on attachment: 209575

Committed r154573: <http://trac.webkit.org/changeset/154573>
Comment 4 WebKit Commit Bot 2013-08-25 02:53:07 PDT 
All reviewed patches have been landed.  Closing bug.