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

Unified Diff: content/browser/storage_partition_impl.cc

Issue 2594723002: Count number of origins with data affected by clearing "cookies and site data". (Closed)
Patch Set: Add comments for issues with incomplete data deletion 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: content/browser/storage_partition_impl.cc
diff --git a/content/browser/storage_partition_impl.cc b/content/browser/storage_partition_impl.cc
index 931e7447e917ce8f885e56cc2ab369c7ff96cbea..e4dbe31e01c70b2eeedaf547e0ad5b0291b1f82a 100644
--- a/content/browser/storage_partition_impl.cc
+++ b/content/browser/storage_partition_impl.cc
@@ -171,8 +171,10 @@ void OnSessionStorageUsageInfo(
for (size_t i = 0; i < infos.size(); ++i) {
if (!origin_matcher.is_null() &&
!origin_matcher.Run(infos[i].origin, special_storage_policy.get())) {
+ LOG(ERROR) << "skip session storage: " << infos[i].origin;
continue;
}
+ LOG(ERROR) << "delete session storage: " << infos[i].origin;
dom_storage_context->DeleteSessionStorage(infos[i]);
}
@@ -721,7 +723,7 @@ StoragePartitionImpl::QuotaManagedDataDeletionHelper::ClearOriginsOnIOThread(
callback.Run();
return;
}
-
+ for(auto origin: origins) LOG(ERROR) << origin.spec();
size_t* deletion_task_count = new size_t(0u);
(*deletion_task_count)++;
for (std::set<GURL>::const_iterator origin = origins.begin();

Powered by Google App Engine
This is Rietveld 408576698