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

Unified Diff: content/browser/service_worker/service_worker_database_unittest.cc

Issue 633873002: Service Worker: Respect the "clear on exit" content setting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@contentsettings
Patch Set: just clear in dtor Created 6 years, 2 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: content/browser/service_worker/service_worker_database_unittest.cc
diff --git a/content/browser/service_worker/service_worker_database_unittest.cc b/content/browser/service_worker/service_worker_database_unittest.cc
index e8fd82cb200450fd54bb58b1779f4e8e168b37c6..83d45893637ab1a98c2c82dc1cff758e1237e05c 100644
--- a/content/browser/service_worker/service_worker_database_unittest.cc
+++ b/content/browser/service_worker/service_worker_database_unittest.cc
@@ -995,9 +995,11 @@ TEST(ServiceWorkerDatabaseTest, DeleteAllDataForOrigin) {
database->WriteRegistration(
data3, resources3, &deleted_version_id, &newly_purgeable_resources));
+ std::set<GURL> origin_to_delete;
+ origin_to_delete.insert(GURL(origin1));
michaeln 2014/10/08 01:22:27 origin1 is of type GURL so no need to construct a
falken 2014/10/08 02:36:44 Oops, that was leftover from trying to make the se
EXPECT_EQ(ServiceWorkerDatabase::STATUS_OK,
- database->DeleteAllDataForOrigin(origin1,
- &newly_purgeable_resources));
+ database->DeleteAllDataForOrigins(origin_to_delete,
+ &newly_purgeable_resources));
// |origin1| should be removed from the unique origin list.
std::set<GURL> unique_origins;

Powered by Google App Engine
This is Rietveld 408576698