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

Unified Diff: chrome/browser/browsing_data_remover.cc

Issue 9358050: BrowsingDataRemover: Enable origin-based deletion for quota-managed data. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebasing. Grrr. Created 8 years, 10 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 | « no previous file | chrome/browser/browsing_data_remover_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data_remover.cc b/chrome/browser/browsing_data_remover.cc
index fe623d2e5148cd6055a9b473b529518c398b9c7b..9130f872af24a570e407ce00f6449248ed7b0909 100644
--- a/chrome/browser/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data_remover.cc
@@ -291,9 +291,6 @@ void BrowsingDataRemover::RemoveImpl(int remove_mask,
if (remove_mask & REMOVE_INDEXEDDB || remove_mask & REMOVE_WEBSQL ||
remove_mask & REMOVE_APPCACHE || remove_mask & REMOVE_FILE_SYSTEMS) {
- // TODO(mkwst): At the moment, we don't have the ability to pass a mask into
- // QuotaManager. Until then, we'll clear all quota-managed data types if any
- // ought to be cleared.
if (!quota_manager_)
quota_manager_ = content::BrowserContext::GetQuotaManager(profile_);
waiting_for_clear_quota_managed_data_ = true;
@@ -601,6 +598,8 @@ void BrowsingDataRemover::OnGotQuotaManagedOrigins(
for (origin = origins.begin(); origin != origins.end(); ++origin) {
if (special_storage_policy_->IsStorageProtected(origin->GetOrigin()))
continue;
+ if (!remove_origin_.is_empty() && remove_origin_ != origin->GetOrigin())
+ continue;
++quota_managed_origins_to_delete_count_;
quota_manager_->DeleteOriginData(
origin->GetOrigin(), type,
« no previous file with comments | « no previous file | chrome/browser/browsing_data_remover_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698