Drag-and-drop: Move startDrag out of WebView/RenderView.

third_party/WebKit/Source/web/ChromeClientImpl.cpp

Index: third_party/WebKit/Source/web/ChromeClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/ChromeClientImpl.cpp b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
index 9a15330d50765bcedde5fba60c91fe238ad266b7..2525bf67bc479160a78886b180451b707d8a87d4 100644
--- a/third_party/WebKit/Source/web/ChromeClientImpl.cpp
+++ b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
@@ -260,7 +260,15 @@ void ChromeClientImpl::startDragging(LocalFrame* frame,
                                      WebDragOperationsMask mask,
                                      const WebImage& dragImage,
                                      const WebPoint& dragImageOffset) {
-  m_webView->startDragging(frame, dragData, mask, dragImage, dragImageOffset);
+  WebLocalFrameImpl* webFrame = WebLocalFrameImpl::fromFrame(frame);
+  WebFrameWidgetBase* frameWidget = webFrame->localRoot()->frameWidget();
+  if (!frameWidget || !frameWidget->client())

[dcheng, 2016/11/07 21:11:00]

Nit: do we need to null-check client or widget here? I'd like to understand how
we can get to this code with either of these being null.

[paulmeyer, 2016/11/07 22:18:13]

Probably not. Removed.

+    return;
+
+  WebReferrerPolicy policy = webFrame->document().referrerPolicy();
+  m_webView->setDoingDragAndDrop(true);
+  frameWidget->client()->startDragging(policy, dragData, mask, dragImage,
+                                       dragImageOffset);
 }
 
 bool ChromeClientImpl::acceptsLoadDrops() const {