Move ShowPopup logic from BrowserActionsContainer to BrowserActionView

chrome/browser/ui/views/toolbar/toolbar_view.cc

Index: chrome/browser/ui/views/toolbar/toolbar_view.cc
diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.cc b/chrome/browser/ui/views/toolbar/toolbar_view.cc
index ae496b6aa1c0565ace60b32687ffa3c10a5df16b..8a64f132363d580cd7708c96f77797e135ba1f4d 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_view.cc
+++ b/chrome/browser/ui/views/toolbar/toolbar_view.cc
@@ -338,7 +338,10 @@ void ToolbarView::ShowPageActionPopup(const extensions::Extension* extension) {
 
 void ToolbarView::ShowBrowserActionPopup(
     const extensions::Extension* extension) {
-  browser_actions_->ShowPopup(extension, true);
+  browser_actions_->ShowPopupForExtension(
+      extension,
+      true,  // Grant active tab permissions.
+      false);  // Don't override the current view.

[Peter Kasting, 2014/07/26 02:33:20]

Nit: I'm not a huge fan of comments on function args.  If the meaning would be
clear enough (possibly with a reference to the function declaration) to the
typical reader, we don't need them.  If not, we may want to consider named enum
values in place of bools.

My normal answer here is to just remove the comments.

[Devlin, 2014/07/29 19:07:15]

I typically view them as providing a reference to the function declaration, to
save any reader from having to look it up (because, as you said, they should be
clear with that).  Tentatively removed the comments, but out of curiosity, would
you feel any better about a shorter "grant_tab_permissions" and "can_override"
(simply documenting which variables each bool is associated with)?

[Peter Kasting, 2014/07/29 19:18:01]

I think those are probably slightly worse than the comments you originally had. 
If we really need to say something because the reader can't look up the function
definition or whatever, it's better to just spell out what we're talking about
than to leave comments that may require the reader to look up the prototype
anyway just to figure them out.

 }
 
 void ToolbarView::ShowAppMenu(bool for_drop) {