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

extensions/renderer/programmatic_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_PROGRAMMATIC_SCRIPT_INJECTOR_H_
 #define EXTENSIONS_RENDERER_PROGRAMMATIC_SCRIPT_INJECTOR_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/values.h"
 #include "extensions/renderer/script_injection.h"
 #include "url/gurl.h"
 
 struct ExtensionMsg_ExecuteCode_Params;
 
 namespace content {
 class RenderFrame;
 }
 
 namespace extensions {
+struct ScriptsRunInfo;
 
 // A ScriptInjector to handle tabs.executeScript().
 class ProgrammaticScriptInjector : public ScriptInjector {
  public:
   ProgrammaticScriptInjector(const ExtensionMsg_ExecuteCode_Params& params,
                              content::RenderFrame* render_frame);
   ~ProgrammaticScriptInjector() override;
 
  private:
   // ScriptInjector implementation.
   UserScript::InjectionType script_type() const override;
   bool ShouldExecuteInMainWorld() const override;
   bool IsUserGesture() const override;
   bool ExpectsResults() const override;
-  bool ShouldInjectJs(UserScript::RunLocation run_location) const override;
-  bool ShouldInjectCss(UserScript::RunLocation run_location) const override;
+  bool ShouldInjectJs(UserScript::RunLocation run_location,
+                      std::map<std::string, std::set<std::string>>&

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

Remember, chromium style discourages non-const refs
(https://codereview.chromium.org/2213603002/diff/200001/extensions/renderer/pr...).
 Use a const & here.

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

Done.

+                          executing_scripts) const override;
+  bool ShouldInjectCss(UserScript::RunLocation run_location,
+                       std::map<std::string, std::set<std::string>>&
+                           executing_scripts) const override;
   PermissionsData::AccessType CanExecuteOnFrame(
       const InjectionHost* injection_host,
       blink::WebLocalFrame* web_frame,
       int tab_id) const override;
   std::vector<blink::WebScriptSource> GetJsSources(
-      UserScript::RunLocation run_location) const override;
+      UserScript::RunLocation run_location,
+      std::map<std::string, std::set<std::string>>* executing_scripts,
+      size_t* num_injected_js_scripts) const override;
   std::vector<blink::WebString> GetCssSources(
-      UserScript::RunLocation run_location) const override;
-  void GetRunInfo(ScriptsRunInfo* scripts_run_info,
-                  UserScript::RunLocation run_location) const override;
+      UserScript::RunLocation run_location,
+      std::map<std::string, std::set<std::string>>* executing_scripts,
+      size_t* num_injected_css_scripts) const override;
   void OnInjectionComplete(std::unique_ptr<base::Value> execution_result,
                            UserScript::RunLocation run_location,
                            content::RenderFrame* render_frame) override;
   void OnWillNotInject(InjectFailureReason reason,
                        content::RenderFrame* render_frame) override;
 
   // Whether it is safe to include information about the URL in error messages.
   bool CanShowUrlInError() const;
 
   // Return the run location for this injector.
@@ skipping 18 matching lines @@
 
   // Whether or not this script injection has finished.
   bool finished_;
 
   DISALLOW_COPY_AND_ASSIGN(ProgrammaticScriptInjector);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_RENDERER_PROGRAMMATIC_SCRIPT_INJECTOR_H_