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

Unified Diff: extensions/browser/management_policy.cc

Issue 2310683002: Remove most ScopedVector usage from c/b/extensions. (Closed)
Patch Set: remove scoped_vector includes Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/browser/management_policy.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/management_policy.cc
diff --git a/extensions/browser/management_policy.cc b/extensions/browser/management_policy.cc
index b09734a908b31b3b1c5c82a4555ed320544aa9d1..7128ba522aa5c09aa33ff018cc4d23f0baad008a 100644
--- a/extensions/browser/management_policy.cc
+++ b/extensions/browser/management_policy.cc
@@ -61,8 +61,10 @@ void ManagementPolicy::UnregisterProvider(Provider* provider) {
providers_.erase(provider);
}
-void ManagementPolicy::RegisterProviders(std::vector<Provider*> providers) {
- providers_.insert(providers.begin(), providers.end());
+void ManagementPolicy::RegisterProviders(
+ const std::vector<std::unique_ptr<Provider>>& providers) {
+ for (const std::unique_ptr<Provider>& provider : providers)
+ providers_.insert(provider.get());
}
bool ManagementPolicy::UserMayLoad(const Extension* extension,
« no previous file with comments | « extensions/browser/management_policy.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698