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

Unified Diff: chrome/browser/visitedlink/visitedlink_unittest.cc

Issue 10096015: Single URL Expires Were Not Being Deleted. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 8 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 | « chrome/browser/visitedlink/visitedlink_master.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/visitedlink/visitedlink_unittest.cc
===================================================================
--- chrome/browser/visitedlink/visitedlink_unittest.cc (revision 132411)
+++ chrome/browser/visitedlink/visitedlink_unittest.cc (working copy)
@@ -263,11 +263,11 @@
// Add more URLs than necessary to trigger this case.
const int kTestDeleteCount = VisitedLinkMaster::kBigDeleteThreshold + 2;
- std::set<GURL> urls_to_delete;
+ history::URLRows urls_to_delete;
for (int32 i = g_test_count; i < g_test_count + kTestDeleteCount; i++) {
GURL url(TestURL(i));
master_->AddURL(url);
- urls_to_delete.insert(url);
+ urls_to_delete.push_back(history::URLRow(url));
}
master_->DeleteURLs(urls_to_delete);
@@ -399,8 +399,8 @@
// Add one more and then delete it.
master_->AddURL(TestURL(g_test_count));
- std::set<GURL> deleted_urls;
- deleted_urls.insert(TestURL(g_test_count));
+ history::URLRows deleted_urls;
+ deleted_urls.push_back(history::URLRow(TestURL(g_test_count)));
master_->DeleteURLs(deleted_urls);
// Wait for the rebuild to complete. The task will terminate the message
@@ -447,8 +447,8 @@
ASSERT_EQ(i + 1, master_->GetUsedCount());
}
- std::set<GURL> deleted_urls;
- deleted_urls.insert(TestURL(0));
+ history::URLRows deleted_urls;
+ deleted_urls.push_back(history::URLRow(TestURL(0)));
// Delete an URL.
master_->DeleteURLs(deleted_urls);
// ... and all of the remaining ones.
« no previous file with comments | « chrome/browser/visitedlink/visitedlink_master.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698