Bug 82420 - [BlackBerry] Use WebCore::PlatformMouseEvent::globalPosition() instead of WebCore::PlatformMouseEvent::globalPos() in WebPage.cpp
Summary: [BlackBerry] Use WebCore::PlatformMouseEvent::globalPosition() instead of Web...
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Leo Yang
Depends on:
Blocks: 73144
  Show dependency treegraph
Reported: 2012-03-27 20:03 PDT by Leo Yang
Modified: 2012-03-28 05:31 PDT (History)
6 users (show)

See Also:

Patch (2.58 KB, patch)
2012-03-27 20:06 PDT, Leo Yang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Leo Yang 2012-03-27 20:03:01 PDT
In PlatformMouseEvent.h the blackberry porting added globalPos() (but not upstreamed yet) for PlatformMouseEvent, but this method is same as globalPosition() that is available for every porting. We should use globalPosition().
Comment 1 Leo Yang 2012-03-27 20:06:35 PDT
Created attachment 134208 [details]
Comment 2 Rob Buis 2012-03-28 04:09:49 PDT
Comment on attachment 134208 [details]

LGTM. Can we remove globalPos()?
Comment 3 Leo Yang 2012-03-28 05:09:30 PDT
(In reply to comment #2)
> (From update of attachment 134208 [details])
> LGTM. Can we remove globalPos()?
Thanks. When I cherry-pick I'll remove it internally.
Comment 4 WebKit Review Bot 2012-03-28 05:31:38 PDT
Comment on attachment 134208 [details]

Clearing flags on attachment: 134208

Committed r112385: <http://trac.webkit.org/changeset/112385>
Comment 5 WebKit Review Bot 2012-03-28 05:31:43 PDT
All reviewed patches have been landed.  Closing bug.