Consolidate ExtensionAction execution code

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 48 matching lines @@
 
   // Convenience method to get the instance for a profile.
   static ExtensionActionAPI* Get(content::BrowserContext* context);
 
   static bool GetBrowserActionVisibility(const ExtensionPrefs* prefs,
                                          const std::string& extension_id);
   static void SetBrowserActionVisibility(ExtensionPrefs* prefs,
                                          const std::string& extension_id,
                                          bool visible);
 
-  // Fires the onClicked event for page_action.
-  static void PageActionExecuted(content::BrowserContext* context,
-                                 const ExtensionAction& page_action,
-                                 int tab_id,
-                                 const std::string& url,
-                                 int button);
-
-  // Fires the onClicked event for browser_action.
-  static void BrowserActionExecuted(content::BrowserContext* context,
-                                    const ExtensionAction& browser_action,
-                                    content::WebContents* web_contents);
-
   // BrowserContextKeyedAPI implementation.
   static BrowserContextKeyedAPIFactory<ExtensionActionAPI>*
       GetFactoryInstance();
 
   // Add or remove observers.
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);
 
+  // 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);
+
   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|.
   void ClearAllValuesForTab(content::WebContents* web_contents);
 
  private:
   friend class BrowserContextKeyedAPIFactory<ExtensionActionAPI>;
 
-  // The DispatchEvent methods forward events to the |profile|'s event router.
-  static void DispatchEventToExtension(content::BrowserContext* context,
-                                       const std::string& extension_id,
-                                       const std::string& event_name,
-                                       scoped_ptr<base::ListValue> event_args);
+  // The DispatchEvent methods forward events to the |context|'s event router.
+  void DispatchEventToExtension(content::BrowserContext* context,
+                                const std::string& extension_id,
+                                const std::string& event_name,
+                                scoped_ptr<base::ListValue> event_args);
 
   // Called when either a browser or page action is executed. Figures out which
   // event to send based on what the extension wants.
-  static void ExtensionActionExecuted(content::BrowserContext* context,
-                                      const ExtensionAction& extension_action,
-                                      content::WebContents* web_contents);
+  void ExtensionActionExecuted(const ExtensionAction& extension_action,
+                               content::WebContents* web_contents);
 
   // BrowserContextKeyedAPI implementation.
   virtual void Shutdown() OVERRIDE;
   static const char* service_name() { return "ExtensionActionAPI"; }
   static const bool kServiceRedirectedInIncognito = true;
 
   ObserverList<Observer> observers_;
 
   content::BrowserContext* browser_context_;
 
@@ skipping 347 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_