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

Unified Diff: components/drive/drive_app_registry.cc

Issue 2613223002: Remove ScopedVector from base::JSONValueConverter (Closed)
Patch Set: Add a TODO Created 3 years, 11 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
Index: components/drive/drive_app_registry.cc
diff --git a/components/drive/drive_app_registry.cc b/components/drive/drive_app_registry.cc
index f6cbc502cdf487607dc11f8af78947e6d531b176..7e62ed2e45c74f79efd2aaae00b993d015704bc3 100644
--- a/components/drive/drive_app_registry.cc
+++ b/components/drive/drive_app_registry.cc
@@ -20,9 +20,10 @@
namespace {
// Add {selector -> app_id} mapping to |map|.
-void AddAppSelectorList(const ScopedVector<std::string>& selectors,
- const std::string& app_id,
- std::multimap<std::string, std::string>* map) {
+void AddAppSelectorList(
+ const std::vector<std::unique_ptr<std::string>>& selectors,
hidehiko 2017/01/10 06:44:36 Ditto for include.
leonhsl(Using Gerrit) 2017/01/10 09:48:13 Done.
+ const std::string& app_id,
+ std::multimap<std::string, std::string>* map) {
for (size_t i = 0; i < selectors.size(); ++i)
map->insert(std::make_pair(*selectors[i], app_id));
}

Powered by Google App Engine
This is Rietveld 408576698