WebView context menu cleanup.

chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_GUEST_VIEW_WEB_VIEW_CHROME_WEB_VIEW_GUEST_DELEGATE_H_
 #define CHROME_BROWSER_GUEST_VIEW_WEB_VIEW_CHROME_WEB_VIEW_GUEST_DELEGATE_H_
 
 #include "chrome/browser/extensions/api/web_view/chrome_web_view_internal_api.h"
 #include "extensions/browser/guest_view/web_view/web_view_guest.h"
 #include "extensions/browser/guest_view/web_view/web_view_guest_delegate.h"
@@ skipping 11 matching lines @@
 namespace extensions {
 
 class ChromeWebViewGuestDelegate : public WebViewGuestDelegate {
  public :
   explicit ChromeWebViewGuestDelegate(WebViewGuest* web_view_guest);
   ~ChromeWebViewGuestDelegate() override;
 
   // WebViewGuestDelegate implementation.
   bool HandleContextMenu(const content::ContextMenuParams& params) override;
   void OnDidInitialize() override;
-  void OnGuestDestroyed() override;
+  void OnGuestDestroyed() override {}

[Fady Samuel, 2015/06/19 18:03:12]

Please get rid of this entirely from delegates if we're no longer using it.

[Devlin, 2015/06/19 18:05:48]

+1.

[paulmeyer, 2015/06/19 18:18:10]

Done.

   void OnShowContextMenu(int request_id, const MenuItemVector* items) override;
 
   WebViewGuest* web_view_guest() const { return web_view_guest_; }
 
  private:
   content::WebContents* guest_web_contents() const {
     return web_view_guest()->web_contents();
   }
 
   // Returns the top level items (ignoring submenus) as Value.
@@ skipping 30 matching lines @@
   // This is used to ensure pending tasks will not fire after this object is
   // destroyed.
   base::WeakPtrFactory<ChromeWebViewGuestDelegate> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeWebViewGuestDelegate);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_GUEST_VIEW_WEB_VIEW_CHROME_WEB_VIEW_GUEST_DELEGATE_H_
-