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

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index c2435b92749f4d355cdfddab4a268706c49d296e..e4e12430999a095ad7de7a229001ccee4a687628 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -27,7 +27,6 @@
 #include "cc/input/browser_controls_state.h"
 #include "cc/resources/shared_bitmap.h"
 #include "content/common/content_export.h"
-#include "content/common/drag_event_source_info.h"
 #include "content/common/frame_message_enums.h"
 #include "content/common/navigation_gesture.h"
 #include "content/common/page_message_enums.h"
@@ -77,10 +76,11 @@ namespace blink {
 class WebApplicationCacheHost;
 class WebDataSource;
 class WebDateTimeChooserCompletion;
-class WebDragData;
 class WebGestureEvent;
 class WebIconURL;
 class WebImage;
+class WebPeerConnection00Handler;
+class WebPeerConnection00HandlerClient;

[nasko, 2016/11/07 23:44:27]

Are those part of the removed header?

[paulmeyer, 2016/11/08 17:07:29]

Actually, I don't know where those came from. Removed.

 class WebMouseEvent;
 class WebSpeechRecognizer;
 class WebStorageNamespace;
@@ -313,11 +313,6 @@ class CONTENT_EXPORT RenderViewImpl
   void setStatusText(const blink::WebString& text) override;
   void setMouseOverURL(const blink::WebURL& url) override;
   void setKeyboardFocusURL(const blink::WebURL& url) override;
-  void startDragging(blink::WebLocalFrame* frame,
-                     const blink::WebDragData& data,
-                     blink::WebDragOperationsMask mask,
-                     const blink::WebImage& image,
-                     const blink::WebPoint& imageOffset) override;
   bool acceptsLoadDrops() override;
   void focusNext() override;
   void focusPrevious() override;
@@ -429,6 +424,7 @@ class CONTENT_EXPORT RenderViewImpl
   friend class RenderViewImplTest;
   friend class RenderViewTest;
   friend class RendererAccessibilityTest;
+  friend class RenderWidget;
 
   // TODO(nasko): Temporarily friend RenderFrameImpl, so we don't duplicate
   // utility functions needed in both classes, while we move frame specific
@@ -497,8 +493,6 @@ class CONTENT_EXPORT RenderViewImpl
   void RenderWidgetFocusChangeComplete() override;
   bool DoesRenderWidgetHaveTouchEventHandlersAt(
       const gfx::Point& point) const override;
-  bool RenderWidgetWillHandleGestureEvent(
-      const blink::WebGestureEvent& event) override;
   bool RenderWidgetWillHandleMouseEvent(
       const blink::WebMouseEvent& event) override;
 
@@ -853,11 +847,6 @@ class CONTENT_EXPORT RenderViewImpl
   // is fine.
   base::ObserverList<RenderViewObserver> observers_;
 
-  // This field stores drag/drop related info for the event that is currently
-  // being handled. If the current event results in starting a drag/drop
-  // session, this info is sent to the browser along with other drag/drop info.
-  DragEventSourceInfo possible_drag_event_info_;
-
   // NOTE: stats_collection_observer_ should be the last members because their
   // constructors call the AddObservers method of RenderViewImpl.
   std::unique_ptr<StatsCollectionObserver> stats_collection_observer_;