Browser Plugin: Simplify NewWindow code

content/browser/browser_plugin/browser_plugin_guest.h

Index: content/browser/browser_plugin/browser_plugin_guest.h
diff --git a/content/browser/browser_plugin/browser_plugin_guest.h b/content/browser/browser_plugin/browser_plugin_guest.h
index 605cafca7edd9652bd7f986326d9228931ea6238..5df45b493e683915e8a4936074d7d5f6c81ac5f4 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.h
+++ b/content/browser/browser_plugin/browser_plugin_guest.h
@@ -195,9 +195,6 @@ class CONTENT_EXPORT BrowserPluginGuest : public NotificationObserver,
   // Exposes the protected web_contents() from WebContentsObserver.
   WebContentsImpl* GetWebContents();
 
-  // Kill the guest process.
-  void Terminate();

[Charlie Reis, 2013/04/04 21:48:46]

Wow, how did that stick around unimplemented?  Thanks for catching the dead
code.

[Fady Samuel, 2013/04/04 22:35:15]

It wasn't getting called.

-
   // Overridden in tests.
   virtual void SetDamageBuffer(
       const BrowserPluginHostMsg_ResizeGuest_Params& params);
@@ -376,10 +373,6 @@ class CONTENT_EXPORT BrowserPluginGuest : public NotificationObserver,
 #if defined(OS_MACOSX)
   void OnShowPopup(const ViewHostMsg_ShowPopup_Params& params);
 #endif
-  void OnShowView(int route_id,
-                  WindowOpenDisposition disposition,
-                  const gfx::Rect& initial_bounds,
-                  bool user_gesture);
   void OnShowWidget(int route_id, const gfx::Rect& initial_pos);
   // Overriden in tests.
   virtual void OnTakeFocus(bool reverse);