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

chrome/browser/ui/content_settings/content_setting_bubble_model.h

Index: chrome/browser/ui/content_settings/content_setting_bubble_model.h
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model.h b/chrome/browser/ui/content_settings/content_setting_bubble_model.h
index eabc97b4bdbcd2f1a44b422e92947418f7a61054..8b211b626fd1be4e0df21d9ad661100fe3071e45 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model.h
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model.h
@@ -17,18 +17,29 @@
 #include "googleurl/src/gurl.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 
-class Browser;
+class BrowserShowContentRelated;
 class Profile;
 class TabContentsWrapper;
 
+class ContentSettingBubbleModelDelegate {
+ public:
+  // Shows the cookies collected in the tab contents wrapper.
+  virtual void ShowCollectedCookiesDialog(TabContentsWrapper* contents) = 0;
+
+  // Shows the Content Settings page for a given content type.
+  virtual void ShowContentSettingsPage(ContentSettingsType type) = 0;
+};

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

Add a protected virtual destructor to make it clear the model doesn't own the
delegate.

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

Done.

+
 // This model provides data for ContentSettingBubble, and also controls
 // the action triggered when the allow / block radio buttons are triggered.
 class ContentSettingBubbleModel : public content::NotificationObserver {
  public:
   virtual ~ContentSettingBubbleModel();
 
+  typedef ContentSettingBubbleModelDelegate Delegate;
+
   static ContentSettingBubbleModel* CreateContentSettingBubbleModel(
-      Browser* browser,
+      Delegate* delegate,
       TabContentsWrapper* tab_contents,
       Profile* profile,
       ContentSettingsType content_type);