Convert drag event position to root frame.

third_party/WebKit/Source/web/WebViewImpl.cpp

Index: third_party/WebKit/Source/web/WebViewImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebViewImpl.cpp b/third_party/WebKit/Source/web/WebViewImpl.cpp
index f3691ab0d9df068f3dad7bc7368733ce2a5d264c..bfe73298fc9b225ee9d9ee7b3180dc9b56cd43e4 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -3709,6 +3709,8 @@ void WebViewImpl::dragTargetDrop(const WebDragData& webDragData, const WebPoint&
     const WebPoint& screenPoint,
     int modifiers)
 {
+    WebPoint clientPointInRootFrame(convertDragEventPointToRootFrame(mainFrameImpl()->frameView(), IntPoint(clientPoint.x, clientPoint.y)));

[bokan, 2016/10/04 23:47:23]

no need for the helper function, just call
page()->frameHost().visualViewport().viewportToRootFrame(clientPoint). This
misses out on DevTools emulation adjustments but that shouldn't be a problem on
Android.

Also, please rename clientPoint to pointInViewport.

[hush (inactive), 2016/10/05 00:24:33]

Done.

+
     DCHECK(m_currentDragData);
     m_currentDragData = DataObject::create(webDragData);
 
@@ -3730,7 +3732,7 @@ void WebViewImpl::dragTargetDrop(const WebDragData& webDragData, const WebPoint&
     m_currentDragData->setModifiers(modifiers);
     DragData dragData(
         m_currentDragData.get(),
-        clientPoint,
+        clientPointInRootFrame,
         screenPoint,
         static_cast<DragOperation>(m_operationsAllowed));
 
@@ -3769,10 +3771,12 @@ WebDragOperation WebViewImpl::dragTargetDragEnterOrOver(const WebPoint& clientPo
 {
     DCHECK(m_currentDragData);
 
+    WebPoint clientPointInRootFrame(convertDragEventPointToRootFrame(mainFrameImpl()->frameView(), IntPoint(clientPoint.x, clientPoint.y)));
+
     m_currentDragData->setModifiers(modifiers);
     DragData dragData(
         m_currentDragData.get(),
-        clientPoint,
+        clientPointInRootFrame,
         screenPoint,
         static_cast<DragOperation>(m_operationsAllowed));