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

Unified Diff: components/content_settings/core/browser/host_content_settings_map.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: components/content_settings/core/browser/host_content_settings_map.cc
diff --git a/components/content_settings/core/browser/host_content_settings_map.cc b/components/content_settings/core/browser/host_content_settings_map.cc
index 4ebb241f3843046d1966fcd6115e62ae5751605d..6e7c0c9d8e180dfa941992becbfb04c892a1ab4c 100644
--- a/components/content_settings/core/browser/host_content_settings_map.cc
+++ b/components/content_settings/core/browser/host_content_settings_map.cc
@@ -725,6 +725,21 @@ void HostContentSettingsMap::ClearSettingsForOneType(
FlushLossyWebsiteSettings();
}
+std::vector<GURL> HostContentSettingsMap::GetSitesWithSettingsInPrefProvider(
msramek 2017/01/09 12:54:44 The website settings that are part of REMOVE_SITE_
dullweber 2017/01/09 16:05:46 Done.
+ ContentSettingsType content_type) {
+ std::set<GURL> origins;
+
+ std::unique_ptr<content_settings::RuleIterator> rule_iterator(
+ pref_provider_->GetRuleIterator(content_type, std::string(), false));
+ if (rule_iterator) {
+ while (rule_iterator->HasNext()) {
+ content_settings::Rule rule = rule_iterator->Next();
+ origins.insert(GURL(rule.primary_pattern.ToString()));
+ }
+ }
+ return std::vector<GURL>(origins.begin(), origins.end());
+}
+
void HostContentSettingsMap::ClearSettingsForOneTypeWithPredicate(
ContentSettingsType content_type,
const base::Callback<bool(const ContentSettingsPattern& primary_pattern,

Powered by Google App Engine
This is Rietveld 408576698