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

chrome/browser/ui/views/location_bar/location_bar_view.h

Index: chrome/browser/ui/views/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index 94777a274bdf0ab48da61fa8b26a15a6ebd0bcc2..6b5da67fc5d042eb029ab57b1b99354b039fe170 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -33,7 +33,11 @@
 #endif
 
 class Browser;
+class BrowserShowContentRelated;

[sky, 2012/03/05 16:30:56]

Can you remove some of these now?

[altimofeev, 2012/03/07 21:01:39]

Done.

+class BrowserShowPageInfo;
+class ContentSettingBubbleModelDelegate;
 class ContentSettingImageView;
+class CommandUpdater;
 class EVBubbleView;
 class ExtensionAction;
 class GURL;
@@ -41,11 +45,18 @@ class InstantController;
 class KeywordHintView;
 class LocationIconView;
 class PageActionWithBadgeView;
+class PageActionImageView;
+class Profile;
 class SelectedKeywordView;
 class StarView;
 class TabContentsWrapper;
 class TemplateURLService;
 
+namespace views {
+class BubbleDelegateView;
+class Widget;
+}
+
 #if defined(OS_WIN) || defined(USE_AURA)
 class SuggestedTextView;
 #endif
@@ -85,6 +96,25 @@ class LocationBarView : public LocationBar,
     // Returns the InstantController, or NULL if there isn't one.
     virtual InstantController* GetInstant() = 0;
 
+    // Creates Widget for the given delegate.
+    virtual views::Widget* CreateViewsBubble(
+        views::BubbleDelegateView* bubble_delegate) = 0;
+
+    // Creates PageActionImageView. Caller gets an ownership.
+    virtual PageActionImageView* CreatePageActionImageView(
+        LocationBarView* owner,
+        ExtensionAction* action) = 0;
+
+    // Returns ContentSettingBubbleModelDelegate.
+    virtual ContentSettingBubbleModelDelegate*
+        GetContentSettingBubbleModelDelegate() = 0;
+
+    // Shows page information in the given web contents.
+    virtual void ShowPageInfo(content::WebContents* web_contents,
+                              const GURL& url,
+                              const content::SSLStatus& ssl,
+                              bool show_history) = 0;
+
     // Called by the location bar view when the user starts typing in the edit.
     // This forces our security style to be UNKNOWN for the duration of the
     // editing.
@@ -111,10 +141,12 @@ class LocationBarView : public LocationBar,
     APP_LAUNCHER
   };
 
-  LocationBarView(Browser* browser,
+  LocationBarView(Profile* profile,
+                  CommandUpdater* command_updater,
                   ToolbarModel* model,
                   Delegate* delegate,
                   Mode mode);
+
   virtual ~LocationBarView();
 
   void Init();
@@ -133,7 +165,11 @@ class LocationBarView : public LocationBar,
   // saved state that the tab holds.
   void Update(const content::WebContents* tab_for_state_restoring);
 
-  Browser* browser() const { return browser_; }
+  // Returns corresponding profile.
+  Profile* profile() const { return profile_; }
+
+  // Returns the delegate.
+  Delegate* delegate() const { return delegate_; }
 
   // Sets |preview_enabled| for the PageAction View associated with this
   // |page_action|. If |preview_enabled| is true, the view will display the
@@ -348,8 +384,8 @@ class LocationBarView : public LocationBar,
   // The Autocomplete Edit field.
   scoped_ptr<OmniboxView> location_entry_;
 
-  // The Browser object that corresponds to this View.
-  Browser* browser_;
+  Profile* profile_;
+  CommandUpdater* command_updater_;
 
   // The model.
   ToolbarModel* model_;