Action box menu

chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h

Index: chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h
diff --git a/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h b/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h
index ac71d854e45ca219c529268a86c85c741d4b4e6b..9df7514d6dd8568dbf09aa5826bb57a9c310dc02 100644
--- a/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h
+++ b/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h
@@ -13,6 +13,7 @@
 #include "base/sequenced_task_runner_helpers.h"
 #include "ui/views/controls/menu/menu_delegate.h"
 
+class Browser;
 class BrowserActionsContainer;
 class BrowserActionView;
 
@@ -34,6 +35,7 @@ class BrowserActionOverflowMenuController : public views::MenuDelegate {
 
   BrowserActionOverflowMenuController(
       BrowserActionsContainer* owner,
+      Browser* browser,
       views::MenuButton* menu_button,
       const std::vector<BrowserActionView*>& views,
       int start_index);
@@ -88,6 +90,9 @@ class BrowserActionOverflowMenuController : public views::MenuDelegate {
   // A pointer to the browser action container that owns the overflow menu.
   BrowserActionsContainer* owner_;
 
+  // Browser associated with BrowserActionOverflowMenuController.

[Peter Kasting, 2012/08/03 23:17:42]

Nit: This comment doesn't add anything as-is (in fact it's kind of confusing
considering BrowserActionOverflowMenuController is the name of this class).

[yefimt, 2012/08/07 00:47:06]

Done.

+  Browser* browser_;
+
   // The observer, may be null.
   Observer* observer_;