Index: chrome/browser/ui/views/toolbar/browser_action_view.h |
diff --git a/chrome/browser/ui/views/toolbar/browser_action_view.h b/chrome/browser/ui/views/toolbar/browser_action_view.h |
index 16a70149e2229a78bc33011b663b7aff0c84ca67..cc3cd679b34f5235424ca201ddd7fecade056d59 100644 |
--- a/chrome/browser/ui/views/toolbar/browser_action_view.h |
+++ b/chrome/browser/ui/views/toolbar/browser_action_view.h |
@@ -56,7 +56,7 @@ class BrowserActionView : public views::MenuButton, |
// Returns the view of the browser actions overflow menu to use as a |
// reference point for a popup when this view isn't visible. |
- virtual views::View* GetOverflowReferenceView() = 0; |
+ virtual views::MenuButton* GetOverflowReferenceView() = 0; |
// Sets the delegate's active popup owner to be |popup_owner|. |
virtual void SetPopupOwner(BrowserActionView* popup_owner) = 0; |
@@ -134,11 +134,6 @@ class BrowserActionView : public views::MenuButton, |
virtual scoped_ptr<views::LabelButtonBorder> CreateDefaultBorder() const |
OVERRIDE; |
- // Notifications when to set button state to pushed/not pushed (for when the |
- // popup/context menu is hidden or shown by the container). |
- void SetButtonPushed(); |
- void SetButtonNotPushed(); |
- |
// Whether the browser action is enabled on this tab. Note that we cannot use |
// the built-in views enabled/SetEnabled because disabled views do not |
// receive drag events. |
@@ -175,6 +170,10 @@ class BrowserActionView : public views::MenuButton, |
virtual void OnWillShowContextMenus() OVERRIDE; |
virtual void OnContextMenuDone() OVERRIDE; |
+ // Sets whether or not a menu (context menu or extension popup) is visible, |
+ // and updates the button state to match. |
+ void SetMenuVisible(bool menu_visible); |
+ |
// The controller for this ExtensionAction view. |
scoped_ptr<ExtensionActionViewController> view_controller_; |