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

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: Latest master 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 86dc2fe6c25f029feba97e5c3886d9faddeee971..c92f6ee0e8e3c45edd043ae7cf379973ef0564d6 100644
--- a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
+++ b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
@@ -65,10 +65,13 @@ extern NSString* const kBrowserActionVisibilityChangedNotification;
@property(readonly, nonatomic) BrowserActionsContainerView* containerView;
// 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