Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(189)

Unified Diff: chrome/browser/ui/cocoa/extensions/browser_actions_controller.h

Issue 766263003: [Extension Toolbar] Refactor and finish pop out logic for actions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sky's + OWNERS Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
diff --git a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
index d6cd4e0133e2a530568f6c703c06a100b5ca01c8..32cf1c8547e6beb48c291cb44502781a33618a2a 100644
--- a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
+++ b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
@@ -66,10 +66,13 @@ extern NSString* const kBrowserActionVisibilityChangedNotification;
@property(readonly, nonatomic) Browser* browser;
// Initializes the controller given the current browser and container view that
-// will hold the browser action buttons.
+// will hold the browser action buttons. If |mainController| is nil, the created
+// BrowserActionsController will be the main controller; otherwise (if this is
+// for the overflow menu), |mainController| should be controller of the main bar
+// for the |browser|.
- (id)initWithBrowser:(Browser*)browser
containerView:(BrowserActionsContainerView*)container
- isOverflow:(BOOL)isOverflow;
+ mainController:(BrowserActionsController*)mainController;
// Update the display of all buttons.
- (void)update;

Powered by Google App Engine
This is Rietveld 408576698