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

Unified Diff: chrome/browser/importer/firefox_importer_browsertest.cc

Issue 2305813002: Remove ScopedVector from importer. (Closed)
Patch Set: rebase 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
Index: chrome/browser/importer/firefox_importer_browsertest.cc
diff --git a/chrome/browser/importer/firefox_importer_browsertest.cc b/chrome/browser/importer/firefox_importer_browsertest.cc
index 874498c92df530f08e413c17f23067ef6bda554d..f1dcd46a3a2326a87df9a37a224a941ad7bb04a6 100644
--- a/chrome/browser/importer/firefox_importer_browsertest.cc
+++ b/chrome/browser/importer/firefox_importer_browsertest.cc
@@ -218,7 +218,7 @@ class FirefoxObserver : public ProfileWriter,
}
}
- void AddKeywords(ScopedVector<TemplateURL> template_urls,
+ void AddKeywords(TemplateURLService::OwnedTemplateURLVector template_urls,
bool unique_on_host_and_path) override {
for (size_t i = 0; i < template_urls.size(); ++i) {
Peter Kasting 2016/09/01 22:55:41 Nit: While here: This loop (and the inner one, I t
Avi (use Gerrit) 2016/09/01 23:28:33 Done.
// The order might not be deterministic, look in the expected list for

Powered by Google App Engine
This is Rietveld 408576698