Re-factor location bar/toolbar code to get rid of the browser dependency.

chrome/browser/ui/browser.h

Index: chrome/browser/ui/browser.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index 966b400244a06d406e6be909d0e1898c00746767..c9e1ae4af6d831c21f755386dc3fc3469ffa12d7 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -31,6 +31,7 @@
 #include "chrome/browser/tabs/tab_strip_model_delegate.h"  // TODO(beng): remove
 #include "chrome/browser/tabs/tab_strip_model_observer.h"  // TODO(beng): remove
 #include "chrome/browser/ui/blocked_content/blocked_content_tab_helper_delegate.h"
+#include "chrome/browser/ui/browser_show_actions.h"
 #include "chrome/browser/ui/bookmarks/bookmark_bar.h"
 #include "chrome/browser/ui/bookmarks/bookmark_tab_helper_delegate.h"
 #include "chrome/browser/ui/browser_navigator.h"
@@ -40,6 +41,7 @@
 #include "chrome/browser/ui/search_engines/search_engine_tab_helper_delegate.h"
 #include "chrome/browser/ui/select_file_dialog.h"
 #include "chrome/browser/ui/tab_contents/core_tab_helper_delegate.h"
+#include "chrome/browser/ui/tab_contents/tab_contents_wrapper_container.h"
 #include "chrome/browser/ui/toolbar/toolbar_model.h"
 #include "chrome/common/content_settings.h"
 #include "chrome/common/content_settings_types.h"
@@ -79,7 +81,8 @@ namespace gfx {
 class Point;
 }
 
-class Browser : public TabHandlerDelegate,
+class Browser : public BrowserShowActions,
+                public TabHandlerDelegate,
                 public content::WebContentsDelegate,
                 public CoreTabHelperDelegate,
                 public SearchEngineTabHelperDelegate,
@@ -91,6 +94,7 @@ class Browser : public TabHandlerDelegate,
                 public CommandUpdater::CommandUpdaterDelegate,
                 public content::NotificationObserver,
                 public SelectFileDialog::Listener,
+                public TabContentsWrapperContainer,
                 public TabRestoreServiceObserver,
                 public ProfileSyncServiceObserver,
                 public InstantDelegate {
@@ -403,18 +407,6 @@ class Browser : public TabHandlerDelegate,
     return tab_handler_->GetTabStripModel();
   }
 
-  int tab_count() const;
-  int active_index() const;
-  int GetIndexOfController(
-      const content::NavigationController* controller) const;
-
-  // TODO(dpapad): Rename to GetActiveTabContentsWrapper().
-  TabContentsWrapper* GetSelectedTabContentsWrapper() const;
-  // A convenient version of the above which returns the TCW's WebContents.
-  content::WebContents* GetSelectedWebContents() const;
-  TabContentsWrapper* GetTabContentsWrapperAt(int index) const;
-  // A convenient version of the above which returns the TCW's WebContents.
-  content::WebContents* GetWebContentsAt(int index) const;
   void ActivateTabAt(int index, bool user_gesture);
   bool IsTabPinned(int index) const;
   bool IsTabDiscarded(int index) const;
@@ -572,10 +564,7 @@ class Browser : public TabHandlerDelegate,
   void SavePage();
   void ViewSelectedSource();
   void ShowFindBar();
-  void ShowPageInfo(content::WebContents* web_contents,
-                    const GURL& url,
-                    const content::SSLStatus& ssl,
-                    bool show_history);
+  // Also see ShowPageInfo method (part of BrowserShowActions interface).
 
   // Returns true if the Browser supports the specified feature. The value of
   // this varies during the lifetime of the browser. For example, if the window
@@ -640,8 +629,6 @@ class Browser : public TabHandlerDelegate,
   void ShowAboutConflictsTab();
   void ShowBrokenPageTab(content::WebContents* contents);
   void ShowOptionsTab(const std::string& sub_page);
-  // Shows the Content Settings page for a given content type.
-  void ShowContentSettingsPage(ContentSettingsType content_type);
   void OpenClearBrowsingDataDialog();
   void OpenOptionsDialog();
   void OpenPasswordManager();
@@ -782,11 +769,27 @@ class Browser : public TabHandlerDelegate,
                                   content::PageTransition transition,
                                   bool user_initiated);
 
-  // Shows the cookies collected in the tab contents wrapper.
-  void ShowCollectedCookiesDialog(TabContentsWrapper* wrapper);
-
   // Interface implementations ////////////////////////////////////////////////
 
+  // Overridden from BrowserShowActions:
+  virtual void ShowCollectedCookiesDialog(TabContentsWrapper* wrapper) OVERRIDE;
+  virtual void ShowContentSettingsPage(
+      ContentSettingsType content_type) OVERRIDE;
+  virtual void ShowPageInfo(content::WebContents* web_contents,
+                            const GURL& url,
+                            const content::SSLStatus& ssl,
+                            bool show_history) OVERRIDE;
+
+  // Overriddent from TabContentsWrapperContainer:
+  virtual int tab_count() const OVERRIDE;
+  virtual int active_index() const OVERRIDE;
+  virtual int GetIndexOfController(
+      const content::NavigationController* controller) const OVERRIDE;
+  virtual TabContentsWrapper* GetSelectedTabContentsWrapper() const OVERRIDE;
+  virtual content::WebContents* GetSelectedWebContents() const OVERRIDE;
+  virtual TabContentsWrapper* GetTabContentsWrapperAt(int index) const OVERRIDE;
+  virtual content::WebContents* GetWebContentsAt(int index) const OVERRIDE;
+
   // Overridden from content::PageNavigator:
   virtual content::WebContents* OpenURL(
       const content::OpenURLParams& params) OVERRIDE;