Title: [142364] trunk/Source/WebKit/blackberry
- Revision
- 142364
- Author
- [email protected]
- Date
- 2013-02-09 09:54:42 -0800 (Sat, 09 Feb 2013)
Log Message
[BlackBerry] Set mouse document position for mouse event in updateCursor.
https://bugs.webkit.org/show_bug.cgi?id=109094.
Patch by Tiancheng Jiang <[email protected]> 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):
Modified Paths
Diff
Modified: trunk/Source/WebKit/blackberry/Api/WebPage.cpp (142363 => 142364)
--- trunk/Source/WebKit/blackberry/Api/WebPage.cpp 2013-02-09 17:52:19 UTC (rev 142363)
+++ trunk/Source/WebKit/blackberry/Api/WebPage.cpp 2013-02-09 17:54:42 UTC (rev 142364)
@@ -2416,6 +2416,9 @@
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);
}
Modified: trunk/Source/WebKit/blackberry/ChangeLog (142363 => 142364)
--- trunk/Source/WebKit/blackberry/ChangeLog 2013-02-09 17:52:19 UTC (rev 142363)
+++ trunk/Source/WebKit/blackberry/ChangeLog 2013-02-09 17:54:42 UTC (rev 142364)
@@ -1,3 +1,20 @@
+2013-02-09 Tiancheng Jiang <[email protected]>
+
+ [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 <[email protected]>
webpage needs null check in BackingStore::setCurrentBackingStoreOwner
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
