[Extensions] Refactor extension action execution

chrome/browser/extensions/extension_action_runner.h

 // Copyright 2014 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_EXTENSION_ACTION_RUNNER_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_ACTION_RUNNER_H_
 
 #include <stdint.h>
 
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/scoped_observer.h"
+#include "chrome/browser/extensions/extension_action.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "extensions/browser/blocked_action_type.h"
 #include "extensions/browser/extension_registry_observer.h"
 #include "extensions/common/permissions/permissions_data.h"
 #include "extensions/common/user_script.h"
 
 namespace content {
 class BrowserContext;
 class WebContents;
 }
 
 namespace IPC {
 class Message;
 }
 
-class ExtensionAction;
-
 namespace extensions {
 class Extension;
 class ExtensionRegistry;
 
 // The provider for ExtensionActions corresponding to scripts which are actively
 // running or need permission.
 class ExtensionActionRunner : public content::WebContentsObserver,
                               public ExtensionRegistryObserver {
  public:
   explicit ExtensionActionRunner(content::WebContents* web_contents);
   ~ExtensionActionRunner() override;
 
-  // Returns the ExtensionActionRunner for the given |web_contents|, or NULL
+  // Returns the ExtensionActionRunner for the given |web_contents|, or null
   // if one does not exist.
   static ExtensionActionRunner* GetForWebContents(
       content::WebContents* web_contents);
 
+  // Executes the action for the given |extension| and returns any further
+  // action (like showing a popup) that should be taken. 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).
+  ExtensionAction::ShowAction RunAction(const Extension* extension,
+                                        bool grant_tab_permissions);
+
+  // Runs any actions that were blocked for the given |extension|. As a
+  // requirement, this will grant activeTab permission to the extension.
+  void RunBlockedActions(const Extension* extension);
+
   // Notifies the ExtensionActionRunner that an extension has been granted
   // active tab permissions. This will run any pending injections for that
   // extension.
   void OnActiveTabPermissionGranted(const Extension* extension);
 
-  // Notifies the ExtensionActionRunner that the action for |extension| has
-  // been clicked, running any pending tasks that were previously shelved.
-  void OnClicked(const Extension* extension);
-
   // Called when a webRequest event for the given |extension| was blocked.
   void OnWebRequestBlocked(const Extension* extension);
 
   // Returns a bitmask of BlockedActionType for the actions that have been
   // blocked for the given extension.
   int GetBlockedActions(const Extension* extension);
 
   // Returns true if the given |extension| has any blocked actions.
   bool WantsToRun(const Extension* extension);
 
@@ skipping 100 matching lines @@
 
   ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
       extension_registry_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionActionRunner);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_ACTION_RUNNER_H_