[Extension Toolbar] Refactor and finish pop out logic for actions

chrome/browser/ui/cocoa/extensions/browser_actions_controller.h

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..a728a8a5ff14838e4151e39c95ce0743bec62885 100644
--- a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
+++ b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
@@ -68,7 +68,7 @@ extern NSString* const kBrowserActionVisibilityChangedNotification;
 // will hold the browser action buttons.
 - (id)initWithBrowser:(Browser*)browser
         containerView:(BrowserActionsContainerView*)container
-           isOverflow:(BOOL)isOverflow;
+       mainController:(BrowserActionsController*)mainController;

[Avi (use Gerrit), 2014/12/12 23:53:59]

Document that parameter. "If this controller is for the overflow, then the
|mainController| parameter is of the controller for the main set of icons. If
|mainController| is nil, this controller is the controller of the main set of
icons" or something like that.

[Devlin, 2014/12/13 00:18:18]

Done.

 
 // Update the display of all buttons.
 - (void)update;