Bug 89245 - [BlackBerry] Remove touch events handling hook for popup
Summary: [BlackBerry] Remove touch events handling hook for popup
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-15 13:57 PDT by Crystal Zhang
Modified: 2012-06-15 16:05 PDT (History)
2 users (show)

See Also:


Attachments
patch (4.62 KB, patch)
2012-06-15 14:07 PDT, Crystal Zhang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Crystal Zhang 2012-06-15 13:57:51 PDT
In the old patches we create a new WebPage for popup, so we need to hook up our own touch handling functions, since we don't use our own WebPage any more, no need to keep the touch handling code, because it just does nothing other than passing out the events which might cause the touch events being handled twice.
Comment 1 Crystal Zhang 2012-06-15 14:07:39 PDT
Created attachment 147896 [details]
patch
Comment 2 Rob Buis 2012-06-15 14:13:25 PDT
Comment on attachment 147896 [details]
patch

Looks good.
Comment 3 WebKit Review Bot 2012-06-15 16:05:32 PDT
Comment on attachment 147896 [details]
patch

Clearing flags on attachment: 147896

Committed r120505: <http://trac.webkit.org/changeset/120505>
Comment 4 WebKit Review Bot 2012-06-15 16:05:36 PDT
All reviewed patches have been landed.  Closing bug.