Add WebDragData to blink::WebView::dragtargetDrop

content/browser/browser_plugin/browser_plugin_guest.cc

Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index d0f1049b8df1423ba0c6748801580955f68ac945..590ad692daa3a77d5e9fb20f0565d566e640c7ad 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -820,7 +820,7 @@ void BrowserPluginGuest::OnDragStatusUpdate(int browser_plugin_instance_id,
       host->DragTargetDragLeave();
       break;
     case blink::WebDragStatusDrop:
-      host->DragTargetDrop(location, location, 0);
+      host->DragTargetDrop(DropData(), location, location, 0);

[lfg, 2016/03/07 21:07:10]

Sanity check, why shouldn't we pass drop_data here?

[hush (inactive), 2016/03/07 21:21:31]

We could, but we don't have to (waste of bytes in IPC?). The drop_data during
"WebDragStatusDrop" is not different from the drop_data in "WebDragStatusEnter".

So passing an invalid dropdata here will make sure we reuse the DropData in
blink::WebViewImpl::dragTargetDrop.
We only pass a valid drop data during dragTargetDrop on Android platform. 

[lfg, 2016/03/07 21:25:23]

lgtm

       if (dragged_url_.is_valid()) {
         delegate_->DidDropLink(dragged_url_);
         dragged_url_ = GURL();