Rework some UserScriptLoader logic in preparation to removing UserScript copy.

extensions/browser/declarative_user_script_master.cc

 // 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.
 
 #include "extensions/browser/declarative_user_script_master.h"
 
 #include "content/public/browser/browser_context.h"
 #include "extensions/browser/extension_user_script_loader.h"
 #include "extensions/browser/user_script_loader.h"
 #include "extensions/browser/web_ui_user_script_loader.h"
@@ skipping 14 matching lines @@
     case HostID::WEBUI:
       loader_.reset(new WebUIUserScriptLoader(browser_context, host_id));
       break;
   }
 }
 
 DeclarativeUserScriptMaster::~DeclarativeUserScriptMaster() {
 }
 
 void DeclarativeUserScriptMaster::AddScript(const UserScript& script) {
-  std::set<UserScript> set;
-  set.insert(script);
-  loader_->AddScripts(set);
+  UserScriptList scripts;
+  scripts.push_back(script);
+  loader_->AddScripts(scripts);

[Devlin, 2016/08/09 23:28:54]

loader_->AddScripts(UserScriptList(1, script));
?

[lazyboy, 2016/08/10 00:03:41]

Done.

 }
 
-void DeclarativeUserScriptMaster::AddScripts(
-    const std::set<UserScript>& scripts,
-    int render_process_id,
-    int render_view_id) {
+void DeclarativeUserScriptMaster::AddScripts(const UserScriptList& scripts,
+                                             int render_process_id,
+                                             int render_view_id) {
   loader_->AddScripts(scripts, render_process_id, render_view_id);
 }
 
 void DeclarativeUserScriptMaster::RemoveScript(const UserScript& script) {
-  std::set<UserScript> set;
-  set.insert(script);
+  std::set<UserScriptIDPair> set;
+  set.insert(UserScriptIDPair(script.id(), script.host_id()));
   loader_->RemoveScripts(set);
 }
 
 void DeclarativeUserScriptMaster::RemoveScripts(
-    const std::set<UserScript>& scripts) {
+    const std::set<UserScriptIDPair>& scripts) {
   loader_->RemoveScripts(scripts);
 }
 
 void DeclarativeUserScriptMaster::ClearScripts() {
   loader_->ClearScripts();
 }
 
 }  // namespace extensions