Prevent duplicate content script injection defined in manifest.json (reland)

extensions/renderer/script_injector.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 EXTENSIONS_RENDERER_SCRIPT_INJECTOR_H_
 #define EXTENSIONS_RENDERER_SCRIPT_INJECTOR_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 33 matching lines @@
   virtual bool ShouldExecuteInMainWorld() const = 0;
 
   // Returns true if the script is running inside a user gesture.
   virtual bool IsUserGesture() const = 0;
 
   // Returns true if the script expects results.
   virtual bool ExpectsResults() const = 0;
 
   // Returns true if the script should inject JS source at the given
   // |run_location|.
-  virtual bool ShouldInjectJs(UserScript::RunLocation run_location) const = 0;
+  virtual bool ShouldInjectJs(UserScript::RunLocation run_location,
+                              std::map<std::string, std::set<std::string>>&
+                                  executing_scripts) const = 0;
 
   // Returns true if the script should inject CSS at the given |run_location|.
-  virtual bool ShouldInjectCss(UserScript::RunLocation run_location) const = 0;
+  virtual bool ShouldInjectCss(UserScript::RunLocation run_location,
+                               std::map<std::string, std::set<std::string>>&
+                                   executing_scripts) const = 0;
 
   // Returns true if the script should execute on the given |frame|.
   virtual PermissionsData::AccessType CanExecuteOnFrame(
       const InjectionHost* injection_host,
       blink::WebLocalFrame* web_frame,
       int tab_id) const = 0;
 
   // Returns the javascript sources to inject at the given |run_location|.
   // Only called if ShouldInjectJs() is true.
   virtual std::vector<blink::WebScriptSource> GetJsSources(
-      UserScript::RunLocation run_location) const = 0;
+      UserScript::RunLocation run_location,
+      std::map<std::string, std::set<std::string>>* executing_scripts,
+      size_t* num_injected_js_scripts) const = 0;
 
   // Returns the css to inject at the given |run_location|.
   // Only called if ShouldInjectCss() is true.
   virtual std::vector<blink::WebString> GetCssSources(
-      UserScript::RunLocation run_location) const = 0;
-
-  // Fill scriptrs run info based on information about injection.
-  virtual void GetRunInfo(
-      ScriptsRunInfo* scripts_run_info,
-      UserScript::RunLocation run_location) const = 0;
+      UserScript::RunLocation run_location,
+      std::map<std::string, std::set<std::string>>* executing_scripts,
+      size_t* num_injected_css_scripts) const = 0;

[Devlin, 2016/09/06 17:21:13]

"scripts" here is a little misleading (css isn't a script).  Let's use the term
"stylesheets"

[catmullings, 2016/09/07 01:00:35]

Done.

 
   // Notifies the script that injection has completed, with a possibly-populated
   // list of results (depending on whether or not ExpectsResults() was true).
   // |render_frame| contains the render frame, or null if the frame was
   // invalidated.
   virtual void OnInjectionComplete(
       std::unique_ptr<base::Value> execution_result,
       UserScript::RunLocation run_location,
       content::RenderFrame* render_frame) = 0;
 
   // Notifies the script that injection will never occur.
   // |render_frame| contains the render frame, or null if the frame was
   // invalidated.
   virtual void OnWillNotInject(InjectFailureReason reason,
                                content::RenderFrame* render_frame) = 0;
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_RENDERER_SCRIPT_INJECTOR_H_