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

extensions/renderer/extension_injection_host.h

 // Copyright 2015 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 EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_
 #define EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_
 
 #include "base/memory/ref_counted.h"
 #include "extensions/common/extension.h"
 #include "extensions/renderer/injection_host.h"
 
 namespace extensions {
+class ExtensionSet;
 
 // A wrapper class that holds an extension and implements the InjectionHost
 // interface.
 class ExtensionInjectionHost : public InjectionHost {
  public:
-  ExtensionInjectionHost(const scoped_refptr<const Extension>& extension);
+  ExtensionInjectionHost(const Extension* extension);
   ~ExtensionInjectionHost() override;
 
+  static scoped_ptr<const ExtensionInjectionHost> Create(

[Devlin, 2015/02/20 22:39:13]

document (in particular, that this can return a null scoped ptr)

[Xi Han, 2015/02/23 16:14:59]

Done.

+      const std::string& extension_id, const ExtensionSet* extensions);
+
  private:
   // InjectionHost:
-  const std::string& GetContentSecurityPolicy() const override;
+  std::string GetContentSecurityPolicy() const override;
   const GURL& url() const override;
   const std::string& name() const override;
   PermissionsData::AccessType CanExecuteOnFrame(
       const GURL& document_url,
       const GURL& top_frame_url,
       int tab_id,
       bool is_declarative) const override;
   bool ShouldNotifyBrowserOfInjection() const override;
 
-  scoped_refptr<const Extension> extension_;
+  const Extension* extension_;

[Devlin, 2015/02/20 22:39:13]

I think it's fine for this to keep a refptr, but whatever's easier. :)

[Xi Han, 2015/02/23 16:14:59]

Acknowledged.

 
   DISALLOW_COPY_AND_ASSIGN(ExtensionInjectionHost);
 };
 
 }  // namespace extesions
 
 #endif  // EXTENSIONS_RENDERER_EXTENSION_INJECTION_HOST_H_