Add WebDragData to blink::WebView::dragtargetDrop

content/browser/renderer_host/render_view_host_impl.h

Index: content/browser/renderer_host/render_view_host_impl.h
diff --git a/content/browser/renderer_host/render_view_host_impl.h b/content/browser/renderer_host/render_view_host_impl.h
index d54b2d7d57322c2ff413483ae9fe0fb8ae766c1d..8e4393175dbc8dcaa3ba820e355c44e46c0caa16 100644
--- a/content/browser/renderer_host/render_view_host_impl.h
+++ b/content/browser/renderer_host/render_view_host_impl.h
@@ -120,9 +120,11 @@ class CONTENT_EXPORT RenderViewHostImpl : public RenderViewHost,
                           blink::WebDragOperationsMask operations_allowed,
                           int key_modifiers) override;
   void DragTargetDragLeave() override;
-  void DragTargetDrop(const gfx::Point& client_pt,
+  void DragTargetDrop(const DropData& drop_data,
+                      const gfx::Point& client_pt,
                       const gfx::Point& screen_pt,
                       int key_modifiers) override;
+  void FilterDropData(DropData* drop_data) override;

[dcheng, 2016/05/24 05:51:50]

Can you put a comment on Enter / Drop that indicates the drop data must be
filtered?

[hush (inactive), 2016/05/24 20:26:09]

Done.

   void EnableAutoResize(const gfx::Size& min_size,
                         const gfx::Size& max_size) override;
   void DisableAutoResize(const gfx::Size& new_size) override;
@@ -345,6 +347,12 @@ class CONTENT_EXPORT RenderViewHostImpl : public RenderViewHost,
   // files without the user's consent.
   void GrantFileAccessFromPageState(const PageState& validated_state);
 
+  // 1. Grants permissions to URL (if any)
+  // 2. Grants permissions to filenames
+  // 3. Grants permissions to file system files.
+  // 4. Register the files with the IsolatedContext.
+  DropData GrantPermissionsToDropData(const DropData& drop_data);

[dcheng, 2016/05/24 05:51:50]

Nit: GrantFileAccessFromDropData, to match the naming convention of the previous
helper method.

[hush (inactive), 2016/05/24 20:26:09]

Done.

+
   // The RenderWidgetHost.
   std::unique_ptr<RenderWidgetHostImpl> render_widget_host_;