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

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

Index: chrome/browser/ui/views/location_bar/click_handler.h
diff --git a/chrome/browser/ui/views/location_bar/click_handler.h b/chrome/browser/ui/views/location_bar/click_handler.h
index ce7dc322b632489726be9e08fefe615536b99012..5a9c400d1c84a892107da9e22bcd622e387b6e47 100644
--- a/chrome/browser/ui/views/location_bar/click_handler.h
+++ b/chrome/browser/ui/views/location_bar/click_handler.h
@@ -8,6 +8,7 @@
 
 #include "base/basictypes.h"
 
+class BrowserShowPageInfo;
 class LocationBarView;
 
 namespace views {
@@ -19,16 +20,18 @@ class View;
 // Info dialog on click, to encapsulate that logic in one place.
 class ClickHandler {
  public:
-  ClickHandler(const views::View* owner, LocationBarView* location_bar);
+  ClickHandler(const views::View* owner,
+               LocationBarView* location_bar,
+               BrowserShowPageInfo* delegate);

[sky, 2012/02/29 22:41:10]

Can this take LocationBarView::Delegate directly?

[altimofeev, 2012/03/01 10:05:39]

I can expose the delegate. Indeed, it will decrease CL size.
Done.

 
   void OnMouseReleased(const views::MouseEvent& event);
 
  private:
   const views::View* owner_;
   LocationBarView* location_bar_;
+  BrowserShowPageInfo* delegate_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(ClickHandler);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_CLICK_HANDLER_H_
-