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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover.cc

Issue 2354843006: [Durable] Clear durable permission when clearing site data. (Closed)
Patch Set: fix test Created 4 years, 3 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: chrome/browser/browsing_data/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover.cc b/chrome/browser/browsing_data/browsing_data_remover.cc
index dd91808d8e33c9dc893b52d21def2fd38380f9eb..b3f2a3fe10ae99d1d1ab7c44dff54741cd081b46 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover.cc
@@ -803,6 +803,13 @@ void BrowsingDataRemover::RemoveImpl(
weak_ptr_factory_.GetWeakPtr())));
}
+ if (remove_mask & REMOVE_DURABLE_PERMISSION) {
+ HostContentSettingsMapFactory::GetForProfile(profile_)
+ ->ClearSettingsForOneTypeWithPredicate(
+ CONTENT_SETTINGS_TYPE_DURABLE_STORAGE,
+ base::Bind(&ForwardPrimaryPatternCallback, same_pattern_filter));
+ }
+
if (remove_mask & REMOVE_LOCAL_STORAGE) {
storage_partition_remove_mask |=
content::StoragePartition::REMOVE_DATA_MASK_LOCAL_STORAGE;
« no previous file with comments | « chrome/browser/browsing_data/browsing_data_remover.h ('k') | chrome/browser/browsing_data/browsing_data_remover_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698