[BlackBerry] Set mouse document position for mouse event in updateCursor.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Feb 2013 17:54:42 +0000 (17:54 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Feb 2013 17:54:42 +0000 (17:54 +0000)
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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@142364 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index 709fe509f00ff3e768f82d74b45b921509ad2baa..81f96a9d36b6e4231f4d6d67a5804aca2df4d405 100644 (file)
@@ -2416,6 +2416,9 @@ void WebPagePrivate::updateCursor()
         m_lastMouseEvent.altKey() ? 0 : KEYMOD_ALT;
 
     BlackBerry::Platform::MouseEvent event(buttonMask, buttonMask, mapToTransformed(m_lastMouseEvent.position()), mapToTransformed(m_lastMouseEvent.globalPosition()), 0, modifiers,  0);
         m_lastMouseEvent.altKey() ? 0 : KEYMOD_ALT;
 
     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);
 }
 
     m_webPage->mouseEvent(event);
 }
 
index a1c456819554fcf0554aa51010f39ac04378f1e8..cf8aee5db4c70ed773ae30e302c8d32af6e50f4b 100644 (file)
@@ -1,3 +1,20 @@
+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>
 
         webpage needs null check in BackingStore::setCurrentBackingStoreOwner
 2013-02-07  Charles Wei  <charles.wei@torchmobile.com.cn>
 
         webpage needs null check in BackingStore::setCurrentBackingStoreOwner