Refactoring: de-couple Extensions from "script injection System" [render side]:3

extensions/renderer/extension_injection_host.cc

Index: extensions/renderer/extension_injection_host.cc
diff --git a/extensions/renderer/extension_injection_host.cc b/extensions/renderer/extension_injection_host.cc
index 5b3c463b0b8e909938696b50f569b3fc3a62ad27..1ce75530fc642a661f3ac461b325c83ad845a256 100644
--- a/extensions/renderer/extension_injection_host.cc
+++ b/extensions/renderer/extension_injection_host.cc
@@ -2,13 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "extensions/common/extension_set.h"
 #include "extensions/common/manifest_handlers/csp_info.h"
 #include "extensions/renderer/extension_injection_host.h"
 
 namespace extensions {
 
 ExtensionInjectionHost::ExtensionInjectionHost(
-    const scoped_refptr<const Extension>& extension)
+    const Extension* extension)
     : InjectionHost(HostID(HostID::EXTENSIONS, extension->id())),
       extension_(extension) {
 }
@@ -16,8 +17,18 @@ ExtensionInjectionHost::ExtensionInjectionHost(
 ExtensionInjectionHost::~ExtensionInjectionHost() {
 }
 
-const std::string& ExtensionInjectionHost::GetContentSecurityPolicy() const {
-  return CSPInfo::GetContentSecurityPolicy(extension_.get());
+// static
+scoped_ptr<const ExtensionInjectionHost> ExtensionInjectionHost::Create(
+    const std::string& extension_id, const ExtensionSet* extensions) {
+  const Extension* extension = extensions->GetByID(extension_id);
+  if (!extension)
+    return scoped_ptr<const ExtensionInjectionHost>();
+  return scoped_ptr<const ExtensionInjectionHost>(
+      new ExtensionInjectionHost(extension));
+}
+
+std::string ExtensionInjectionHost::GetContentSecurityPolicy() const {
+  return CSPInfo::GetContentSecurityPolicy(extension_);
 }
 
 const GURL& ExtensionInjectionHost::url() const {
@@ -38,7 +49,7 @@ PermissionsData::AccessType ExtensionInjectionHost::CanExecuteOnFrame(
   // "content script access" logic.
   if (is_declarative) {
     return extension_->permissions_data()->GetPageAccess(
-        extension_.get(),
+        extension_,
         document_url,
         top_frame_url,
         tab_id,
@@ -46,7 +57,7 @@ PermissionsData::AccessType ExtensionInjectionHost::CanExecuteOnFrame(
         nullptr /* ignore error */);
   } else {
     return extension_->permissions_data()->GetContentScriptAccess(
-        extension_.get(),
+        extension_,
         document_url,
         top_frame_url,
         tab_id,
@@ -61,7 +72,7 @@ bool ExtensionInjectionHost::ShouldNotifyBrowserOfInjection() const {
   // otherwise been affected by the scripts-require-action feature.
   return extension_->permissions_data()->withheld_permissions()->IsEmpty() &&
          PermissionsData::ScriptsMayRequireActionForExtension(
-             extension_.get(),
+             extension_,
              extension_->permissions_data()->active_permissions().get());
 }