Remove TabContentsView methods that just call to the delegate, and have them be part of the delegate interface instead. This is towards making TabContentsView just an interface.

BUG=87702
Review URL: http://codereview.chromium.org/7277027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91989 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 29 insertions, 55 deletions

diff --git a/content/browser/renderer_host/render_view_host_delegate.h b/content/browser/renderer_host/render_view_host_delegate.h
index ee9cd26..044db27 100644
--- a/content/browser/renderer_host/render_view_host_delegate.h
+++ b/content/browser/renderer_host/render_view_host_delegate.h
@@ -25,52 +25,27 @@ class BackgroundContents;
 struct BookmarkNodeData;
 class BookmarkNode;
 struct ContextMenuParams;
-class FilePath;
 class GURL;
-class ListValue;
 struct NativeWebKeyboardEvent;
-class NavigationEntry;
 class Profile;
 struct RendererPreferences;
 class RenderProcessHost;
 class RenderViewHost;
-class ResourceRedirectDetails;
-class ResourceRequestDetails;
 class SkBitmap;
 class TabContents;
 struct ViewHostMsg_CreateWindow_Params;
 struct ViewHostMsg_FrameNavigate_Params;
-struct WebApplicationInfo;
 struct WebDropData;
 struct WebMenuItem;
 class WebKeyboardEvent;
 struct WebPreferences;
 
-namespace base {
-class WaitableEvent;
-}
-
 namespace gfx {
 class Point;
 class Rect;
 class Size;
 }
 
-namespace IPC {
-class Message;
-}
-
-namespace net {
-class CookieList;
-class CookieOptions;
-}
-
-namespace webkit_glue {
-struct FormData;
-struct FormField;
-struct PasswordForm;
-}
-
 //
 // RenderViewHostDelegate
 //
@@ -87,7 +62,6 @@ class RenderViewHostDelegate : public IPC::Channel::Listener {
  public:
   // View ----------------------------------------------------------------------
   // Functions that can be routed directly to a view-specific class.
-
   class View {
    public:
     // The page is trying to open a new page (e.g. a popup window). The window
@@ -171,35 +145,6 @@ class RenderViewHostDelegate : public IPC::Channel::Listener {
     // retrieved by doing a Shift-Tab.
     virtual void TakeFocus(bool reverse) = 0;
 
-    // Notification that the view has lost capture.
-    virtual void LostCapture() = 0;
-
-    // The page wants the hosting window to activate/deactivate itself (it
-    // called the JavaScript window.focus()/blur() method).
-    virtual void Activate() = 0;
-    virtual void Deactivate() = 0;
-
-    // Callback to give the browser a chance to handle the specified keyboard
-    // event before sending it to the renderer.
-    // Returns true if the |event| was handled. Otherwise, if the |event| would
-    // be handled in HandleKeyboardEvent() method as a normal keyboard shortcut,
-    // |*is_keyboard_shortcut| should be set to true.
-    virtual bool PreHandleKeyboardEvent(const NativeWebKeyboardEvent& event,
-                                        bool* is_keyboard_shortcut) = 0;
-
-    // Callback to inform the browser that the renderer did not process the
-    // specified events. This gives an opportunity to the browser to process the
-    // event (used for keyboard shortcuts).
-    virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event) = 0;
-
-    // Notifications about mouse events in this view.  This is useful for
-    // implementing global 'on hover' features external to the view.
-    virtual void HandleMouseMove() = 0;
-    virtual void HandleMouseDown() = 0;
-    virtual void HandleMouseLeave() = 0;
-    virtual void HandleMouseUp() = 0;
-    virtual void HandleMouseActivate() = 0;
-
     // The contents' preferred size changed.
     virtual void UpdatePreferredSize(const gfx::Size& pref_size) = 0;
 
@@ -390,6 +335,35 @@ class RenderViewHostDelegate : public IPC::Channel::Listener {
   // Notification that a worker process has crashed.
   void WorkerCrashed() {}
 
+  // The page wants the hosting window to activate/deactivate itself (it
+  // called the JavaScript window.focus()/blur() method).
+  virtual void Activate() {}
+  virtual void Deactivate() {}
+
+  // Notification that the view has lost capture.
+  virtual void LostCapture() {}
+
+  // Callback to give the browser a chance to handle the specified keyboard
+  // event before sending it to the renderer.
+  // Returns true if the |event| was handled. Otherwise, if the |event| would
+  // be handled in HandleKeyboardEvent() method as a normal keyboard shortcut,
+  // |*is_keyboard_shortcut| should be set to true.
+  virtual bool PreHandleKeyboardEvent(const NativeWebKeyboardEvent& event,
+                                      bool* is_keyboard_shortcut);
+
+  // Callback to inform the browser that the renderer did not process the
+  // specified events. This gives an opportunity to the browser to process the
+  // event (used for keyboard shortcuts).
+  virtual void HandleKeyboardEvent(const NativeWebKeyboardEvent& event) {}
+
+  // Notifications about mouse events in this view.  This is useful for
+  // implementing global 'on hover' features external to the view.
+  virtual void HandleMouseMove() {}
+  virtual void HandleMouseDown() {}
+  virtual void HandleMouseLeave() {}
+  virtual void HandleMouseUp() {}
+  virtual void HandleMouseActivate() {}
+
  protected:
   virtual ~RenderViewHostDelegate() {}
 };