Changeset 142364 in webkit

Timestamp:

Feb 9, 2013 9:54:42 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[BlackBerry] Set mouse document position for mouse event in updateCursor.
​https://bugs.webkit.org/show_bug.cgi?id=109094.

Patch by Tiancheng Jiang <​tijiang@rim.com> on 2013-02-09
Reviewed by Rob Buis.

RIM PR 246976
Internally Reviewed by Genevieve Mak.

BlackBerry::Platform::MouseEvent have document viewport and document
content position as members. When we create the event, we should initial
them as well.

• Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::updateCursor):

Location:

trunk/Source/WebKit/blackberry

Files:

• Api/WebPage.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/Source/WebKit/blackberry/Api/WebPage.cpp

@@ -2417 +2417 @@
 
     BlackBerry::Platform::MouseEvent event(buttonMask, buttonMask, mapToTransformed(m_lastMouseEvent.position()), mapToTransformed(m_lastMouseEvent.globalPosition()), 0, modifiers,  0);
+
+    // We have added document viewport position and document content position as members of the mouse event, when we create the event, we should initial them as well.
+    event.populateDocumentPosition(m_lastMouseEvent.position(), mapFromTransformedViewportToTransformedContents(m_lastMouseEvent.position()));
     m_webPage->mouseEvent(event);
 }

trunk/Source/WebKit/blackberry/ChangeLog

@@ +1 @@
+2013-02-09  Tiancheng Jiang  <tijiang@rim.com>
+
+        [BlackBerry] Set mouse document position for mouse event in updateCursor.
+        https://bugs.webkit.org/show_bug.cgi?id=109094.
+
+        Reviewed by Rob Buis.
+
+        RIM PR 246976
+        Internally Reviewed by Genevieve Mak.
+
+        BlackBerry::Platform::MouseEvent have document viewport and document
+        content position as members. When we create the event, we should initial
+        them as well.
+
+        * Api/WebPage.cpp:
+        (BlackBerry::WebKit::WebPagePrivate::updateCursor):
+
 2013-02-07  Charles Wei  <charles.wei@torchmobile.com.cn>