Action box menu

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

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_ACTION_BOX_BUTTON_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_ACTION_BOX_BUTTON_VIEW_H_
 
 #include "ui/views/controls/button/menu_button.h"
 #include "ui/views/controls/button/menu_button_listener.h"
 
-class ExtensionService;
+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;
 
   // MenuButtonListener
   virtual void OnMenuButtonClicked(View* source,
                                    const gfx::Point& point) OVERRIDE;
 
-  ExtensionService* extension_service_;
+  Browser* browser_;
+  Profile* profile_;
+
+  // Set to true when current page is bookmarked. To passed to action box menu

[msw, 2012/07/25 23:02:03]

nit: "*the* current", "To *be* passed", and "*the* action box".

[yefimt, 2012/07/31 00:10:11]

Done.

+  // when this button is pressed.
+  bool bookmark_state_;

[Peter Kasting, 2012/07/26 20:37:17]

Nit: As in action_box_menu.h, call this |starred_| (and rename the accessors as
needed).

[yefimt, 2012/07/31 00:10:11]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(ActionBoxButtonView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_LOCATION_BAR_ACTION_BOX_BUTTON_VIEW_H_