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

Unified Diff: components/history/core/browser/history_types_unittest.cc

Issue 2906953003: Remove ScopedVector usage from components/history (Closed)
Patch Set: Remove ScopedVector usage from components/history Created 3 years, 6 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 | « components/history/core/browser/history_types.cc ('k') | ios/chrome/browser/tabs/tab_unittest.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/history/core/browser/history_types_unittest.cc
diff --git a/components/history/core/browser/history_types_unittest.cc b/components/history/core/browser/history_types_unittest.cc
index 8641baa33040ef9f8f3f3098f2783a86ce25ed66..09dd46cefb3bf65f6e4af9689858ecccf76e36fc 100644
--- a/components/history/core/browser/history_types_unittest.cc
+++ b/components/history/core/browser/history_types_unittest.cc
@@ -40,14 +40,14 @@ const char kURL2[] = "http://news.google.com/";
void AddSimpleData(QueryResults* results) {
GURL url1(kURL1);
GURL url2(kURL2);
- URLResult result1(url1, base::Time::Now());
- URLResult result2(url1, base::Time::Now());
- URLResult result3(url2, base::Time::Now());
+ std::vector<URLResult> test_vector;
+
+ test_vector.push_back(URLResult(url1, base::Time::Now()));
+ test_vector.push_back(URLResult(url1, base::Time::Now()));
+ test_vector.push_back(URLResult(url2, base::Time::Now()));
// The URLResults are invalid after being inserted.
- results->AppendURLBySwapping(&result1);
- results->AppendURLBySwapping(&result2);
- results->AppendURLBySwapping(&result3);
+ results->SetURLResults(std::move(test_vector));
CheckHistoryResultConsistency(*results);
}
« no previous file with comments | « components/history/core/browser/history_types.cc ('k') | ios/chrome/browser/tabs/tab_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698