Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(744)

Side by Side Diff: extensions/browser/declarative_user_script_master.cc

Issue 2228743002: Rework some UserScriptLoader logic in preparation to removing UserScript copy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: sync @tott Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/browser/declarative_user_script_master.h" 5 #include "extensions/browser/declarative_user_script_master.h"
6 6
7 #include "content/public/browser/browser_context.h" 7 #include "content/public/browser/browser_context.h"
8 #include "extensions/browser/extension_user_script_loader.h" 8 #include "extensions/browser/extension_user_script_loader.h"
9 #include "extensions/browser/user_script_loader.h" 9 #include "extensions/browser/user_script_loader.h"
10 #include "extensions/browser/web_ui_user_script_loader.h" 10 #include "extensions/browser/web_ui_user_script_loader.h"
(...skipping 14 matching lines...) Expand all
25 case HostID::WEBUI: 25 case HostID::WEBUI:
26 loader_.reset(new WebUIUserScriptLoader(browser_context, host_id)); 26 loader_.reset(new WebUIUserScriptLoader(browser_context, host_id));
27 break; 27 break;
28 } 28 }
29 } 29 }
30 30
31 DeclarativeUserScriptMaster::~DeclarativeUserScriptMaster() { 31 DeclarativeUserScriptMaster::~DeclarativeUserScriptMaster() {
32 } 32 }
33 33
34 void DeclarativeUserScriptMaster::AddScript(const UserScript& script) { 34 void DeclarativeUserScriptMaster::AddScript(const UserScript& script) {
35 std::set<UserScript> set; 35 loader_->AddScripts(UserScriptList(1, script));
36 set.insert(script);
37 loader_->AddScripts(set);
38 } 36 }
39 37
40 void DeclarativeUserScriptMaster::AddScripts( 38 void DeclarativeUserScriptMaster::AddScripts(const UserScriptList& scripts,
41 const std::set<UserScript>& scripts, 39 int render_process_id,
42 int render_process_id, 40 int render_view_id) {
43 int render_frame_id) { 41 loader_->AddScripts(scripts, render_process_id, render_view_id);
44 loader_->AddScripts(scripts, render_process_id, render_frame_id);
45 } 42 }
46 43
47 void DeclarativeUserScriptMaster::RemoveScript(const UserScript& script) { 44 void DeclarativeUserScriptMaster::RemoveScript(const UserScript& script) {
48 std::set<UserScript> set; 45 std::set<UserScriptIDPair> set;
49 set.insert(script); 46 set.insert(UserScriptIDPair(script.id(), script.host_id()));
50 loader_->RemoveScripts(set); 47 loader_->RemoveScripts(set);
51 } 48 }
52 49
53 void DeclarativeUserScriptMaster::RemoveScripts( 50 void DeclarativeUserScriptMaster::RemoveScripts(
54 const std::set<UserScript>& scripts) { 51 const std::set<UserScriptIDPair>& scripts) {
55 loader_->RemoveScripts(scripts); 52 loader_->RemoveScripts(scripts);
56 } 53 }
57 54
58 void DeclarativeUserScriptMaster::ClearScripts() { 55 void DeclarativeUserScriptMaster::ClearScripts() {
59 loader_->ClearScripts(); 56 loader_->ClearScripts();
60 } 57 }
61 58
62 } // namespace extensions 59 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698