Make a ShowExtensionActionPopup function

chrome/browser/extensions/api/extension_action/extension_action_api.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_EXTENSION_ACTION_EXTENSION_ACTION_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_EXTENSION_ACTION_EXTENSION_ACTION_API_H_
 
 #include <string>
 
 #include "base/memory/weak_ptr.h"
@@ skipping 69 matching lines @@
 
   // Executes the action of the given |extension| on the |browser|'s active
   // web contents. If |grant_tab_permissions| is true, this will also grant
   // activeTab to the extension (so this should only be done if this is through
   // a direct user action). Returns the action that should be taken.
   ExtensionAction::ShowAction ExecuteExtensionAction(
       const Extension* extension,
       Browser* browser,
       bool grant_active_tab_permissions);
 
+  // Opens the popup for the given |extension| in the given |browser|'s window.
+  // See ExecuteExtensionAction for the definition of
+  // |grant_active_tab_permissions|.

[Finnur, 2014/08/27 11:53:02]

Hmm... I wonder if we should just duplicate the comment for it here, since
|grant_active_tab_permissions| is important to get right (to avoid security
flaws).

[Devlin, 2014/08/27 15:43:22]

Done.

+  bool ShowExtensionActionPopup(const Extension* extension,
+                                Browser* browser,
+                                bool grant_active_tab_permissions);
+
   // Notifies that there has been a change in the given |extension_action|.
   void NotifyChange(ExtensionAction* extension_action,
                     content::WebContents* web_contents,
                     content::BrowserContext* browser_context);
 
   // Clears the values for all ExtensionActions for the tab associated with the
   // given |web_contents| (and signals that page actions changed).
   void ClearAllValuesForTab(content::WebContents* web_contents);
 
   // Notifies that the current set of page actions for |web_contents| has
@@ skipping 373 matching lines @@
 class PageActionGetPopupFunction
     : public extensions::ExtensionActionGetPopupFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("pageAction.getPopup", PAGEACTION_GETPOPUP)
 
  protected:
   virtual ~PageActionGetPopupFunction() {}
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_EXTENSION_ACTION_EXTENSION_ACTION_API_H_