Action box menu

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

Index: chrome/browser/ui/views/location_bar/action_box_button_view.h
diff --git a/chrome/browser/ui/views/location_bar/action_box_button_view.h b/chrome/browser/ui/views/location_bar/action_box_button_view.h
index 522eb4e03120114a357c1a75a1ab0c6baeea9b90..be4455643a709bc79d0bfe7717dd7b4ad84fbf13 100644
--- a/chrome/browser/ui/views/location_bar/action_box_button_view.h
+++ b/chrome/browser/ui/views/location_bar/action_box_button_view.h
@@ -8,18 +8,24 @@
 #include "ui/views/controls/button/menu_button.h"
 #include "ui/views/controls/button/menu_button_listener.h"
 
-class ExtensionService;
+class ActionBoxMenu;

[msw, 2012/07/24 23:41:55]

nit: remove this unused forward decl.

[yefimt, 2012/07/25 21:09:21]

Done.

+class Browser;
+class Profile;
 
 // ActionBoxButtonView displays a plus button with associated menu.
 class ActionBoxButtonView : public views::MenuButton,
                             public views::MenuButtonListener {
  public:
-  explicit ActionBoxButtonView(ExtensionService* extension_service);
+  explicit ActionBoxButtonView(Browser* browser, Profile* profile);
   virtual ~ActionBoxButtonView();
 
   SkColor GetBackgroundColor();
   SkColor GetBorderColor();
 
+  void set_bookmark_state(bool bookmark_state) {
+    bookmark_state_ = bookmark_state;
+  }
+
  private:
   // CustomButton
   virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE;
@@ -28,7 +34,9 @@ class ActionBoxButtonView : public views::MenuButton,
   virtual void OnMenuButtonClicked(View* source,
                                    const gfx::Point& point) OVERRIDE;
 
-  ExtensionService* extension_service_;
+  Browser* browser_;
+  Profile* profile_;
+  bool bookmark_state_;

[msw, 2012/07/24 23:41:55]

nit: explain the significance of bookmark_state_ here.

[yefimt, 2012/07/25 21:09:21]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(ActionBoxButtonView);
 };