Remove dead RenderWidgetHostViewEventHandler::ShowContextMenu()

R=avi@chromium.org

Change-Id: Ia1beb8f5e22c53e38803f44302785d63abe82363
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2391728Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#804264}

content/browser/renderer_host/render_view_host_impl.cc

@@ -538,11 +538,6 @@ WebPreferences RenderViewHostImpl::GetWebkitPreferencesForWidget() {
   return delegate_->GetOrCreateWebPreferences();
 }
 
-void RenderViewHostImpl::ShowContextMenu(RenderFrameHost* render_frame_host,
-                                         const ContextMenuParams& params) {
-  GetDelegate()->GetDelegateView()->ShowContextMenu(render_frame_host, params);
-}
-
 void RenderViewHostImpl::DispatchRenderViewCreated() {
   if (has_notified_about_creation_)
     return;

content/browser/renderer_host/render_view_host_impl.h

@@ -301,9 +301,6 @@ class CONTENT_EXPORT RenderViewHostImpl
   bool IsNeverComposited() override;
   WebPreferences GetWebkitPreferencesForWidget() override;
 
-  void ShowContextMenu(RenderFrameHost* render_frame_host,
-                       const ContextMenuParams& params) override;
-
   // IPC message handlers.
   void OnShowView(int route_id,
                   WindowOpenDisposition disposition,

content/browser/renderer_host/render_widget_host_owner_delegate.h

@@ -18,9 +18,7 @@ class Rect;
 }
 
 namespace content {
-struct ContextMenuParams;
 struct NativeWebKeyboardEvent;
-class RenderFrameHost;
 struct WebPreferences;
 
 //
@@ -79,11 +77,6 @@ class CONTENT_EXPORT RenderWidgetHostOwnerDelegate {
   // between all widgets for the page.
   virtual WebPreferences GetWebkitPreferencesForWidget() = 0;
 
-  // Shows a context menu that is built using the context information
-  // provided in |params|.
-  virtual void ShowContextMenu(RenderFrameHost* render_frame_host,
-                               const ContextMenuParams& params) = 0;
-
  protected:
   virtual ~RenderWidgetHostOwnerDelegate() {}
 };

content/browser/renderer_host/render_widget_host_view_aura.cc

@@ -2241,18 +2241,6 @@ void RenderWidgetHostViewAura::SetTooltipsEnabled(bool enable) {
   }
 }
 
-void RenderWidgetHostViewAura::ShowContextMenu(
-    const ContextMenuParams& params) {
-  // Use RenderViewHostDelegate to get to the WebContentsViewAura, which will
-  // actually show the disambiguation popup.
-  // NOTE: This only works for main frame widgets then, as child frame widgets
-  // don't have an owner delegate and won't get access to the RenderViewHost
-  // here.
-  RenderWidgetHostOwnerDelegate* owner_delegate = host()->owner_delegate();
-  if (owner_delegate)
-    owner_delegate->ShowContextMenu(GetFocusedFrame(), params);
-}
-
 void RenderWidgetHostViewAura::NotifyRendererOfCursorVisibilityState(
     bool is_visible) {
   if (host()->is_hidden() ||

content/browser/renderer_host/render_widget_host_view_aura.h

@@ -341,7 +341,6 @@ class CONTENT_EXPORT RenderWidgetHostViewAura
   RenderFrameHostImpl* GetFocusedFrame() const;
   bool NeedsMouseCapture() override;
   void SetTooltipsEnabled(bool enable) override;
-  void ShowContextMenu(const ContextMenuParams& params) override;
   void Shutdown() override;
 
   bool ShouldVirtualKeyboardOverlayContent() const;

content/browser/renderer_host/render_widget_host_view_event_handler.h

@@ -80,7 +80,6 @@ class CONTENT_EXPORT RenderWidgetHostViewEventHandler
     // Returns whether the widget needs to grab mouse capture to work properly.
     virtual bool NeedsMouseCapture() = 0;
     virtual void SetTooltipsEnabled(bool enable) = 0;
-    virtual void ShowContextMenu(const ContextMenuParams& params) = 0;
     // Sends shutdown request.
     virtual void Shutdown() = 0;
 

content/test/stub_render_widget_host_owner_delegate.h

@@ -26,8 +26,6 @@ class StubRenderWidgetHostOwnerDelegate : public RenderWidgetHostOwnerDelegate {
   bool IsMainFrameActive() override;
   bool IsNeverComposited() override;
   WebPreferences GetWebkitPreferencesForWidget() override;
-  void ShowContextMenu(RenderFrameHost* render_frame_host,
-                       const ContextMenuParams& params) override {}
 };
 
 }  // namespace content