Implement initial chrome.browserAction.openPopup API.

chrome/browser/ui/views/browser_actions_container.h

Index: chrome/browser/ui/views/browser_actions_container.h
diff --git a/chrome/browser/ui/views/browser_actions_container.h b/chrome/browser/ui/views/browser_actions_container.h
index 9e3fc949c58b67c126b73e3f558ee3d9f8a75b6f..db4ae6b929b6ffc0b38781c3176a36cf0346ca9f 100644
--- a/chrome/browser/ui/views/browser_actions_container.h
+++ b/chrome/browser/ui/views/browser_actions_container.h
@@ -259,6 +259,8 @@ class BrowserActionsContainer
       const extensions::Extension* extension) OVERRIDE;
   virtual void BrowserActionMoved(const extensions::Extension* extension,
                                   int index) OVERRIDE;
+  virtual bool BrowserActionShowPopup(
+      const extensions::Extension* extension) OVERRIDE;
   virtual void ModelLoaded() OVERRIDE;
 
   void LoadImages();
@@ -309,8 +311,11 @@ class BrowserActionsContainer
   // for incognito.
   bool ShouldDisplayBrowserAction(const extensions::Extension* extension);
 
-  // Show a popup.
-  void ShowPopup(BrowserActionButton* button,
+  // Show a popup. Returns true if a new popup was shown.
+  bool ShowPopup(BrowserActionButton* button,
+                 ExtensionPopup::ShowAction show_action,
+                 bool should_grant);

[Finnur, 2013/10/17 14:58:34]

Same argument here (one function or two similar functions, not identically
named).

[justinlin, 2013/10/17 18:32:53]

Done.

+  bool ShowPopup(BrowserActionButton* button,
                  ExtensionPopup::ShowAction show_action);
 
   // The vector of browser actions (icons/image buttons for each action). Note