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

extensions/renderer/script_injector.h

Index: extensions/renderer/script_injector.h
diff --git a/extensions/renderer/script_injector.h b/extensions/renderer/script_injector.h
index 2443942e652715c2833e87050925ed716bcd4ebb..2e5234623343c8c8faff8b58d868ce9389bdd36d 100644
--- a/extensions/renderer/script_injector.h
+++ b/extensions/renderer/script_injector.h
@@ -51,10 +51,12 @@ class ScriptInjector {
 
   // 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,
+                              ScriptsRunInfo* scripts_run_info) const = 0;

[Devlin, 2016/08/03 23:34:44]

Let's take this as an opportunity to slightly reduce the footprint here and just
pass in the std::set<GURL> of injected_scripts

[catmullings, 2016/08/09 23:13:41]

Done.

 
   // 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,
+                               ScriptsRunInfo* scripts_run_info) const = 0;
 
   // Returns true if the script should execute on the given |frame|.
   virtual PermissionsData::AccessType CanExecuteOnFrame(
@@ -65,12 +67,14 @@ class ScriptInjector {
   // 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,
+      ScriptsRunInfo* scripts_run_info) const = 0;
 
   // Returns the css to inject at the given |run_location|.
   // Only called if ShouldInjectCss() is true.
   virtual std::vector<std::string> GetCssSources(
-      UserScript::RunLocation run_location) const = 0;
+      UserScript::RunLocation run_location,
+      ScriptsRunInfo* scripts_run_info) const = 0;
 
   // Fill scriptrs run info based on information about injection.
   virtual void GetRunInfo(