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

Unified Diff: components/content_settings/core/browser/content_settings_pref_provider.cc

Issue 2812113004: Write last_modified date to Content Settings in the PrefProvider (Closed)
Patch Set: rebase Created 3 years, 8 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/content_settings_pref_provider.cc
diff --git a/components/content_settings/core/browser/content_settings_pref_provider.cc b/components/content_settings/core/browser/content_settings_pref_provider.cc
index 43738618351c6995cda3dc7c58269c655f3fc680..98ab6245d6c1b8bff0b48c31a428e746b24609e7 100644
--- a/components/content_settings/core/browser/content_settings_pref_provider.cc
+++ b/components/content_settings/core/browser/content_settings_pref_provider.cc
@@ -80,9 +80,10 @@ void PrefProvider::RegisterProfilePrefs(
#endif // !defined(OS_IOS)
}
-PrefProvider::PrefProvider(PrefService* prefs, bool incognito)
- : prefs_(prefs),
- is_incognito_(incognito) {
+PrefProvider::PrefProvider(PrefService* prefs,
+ bool incognito,
+ bool store_last_modified)
+ : prefs_(prefs), is_incognito_(incognito) {
DCHECK(prefs_);
// Verify preferences version.
if (!prefs_->HasPrefPath(prefs::kContentSettingsVersion)) {
@@ -105,7 +106,7 @@ PrefProvider::PrefProvider(PrefService* prefs, bool incognito)
info->type(),
base::MakeUnique<ContentSettingsPref>(
info->type(), prefs_, &pref_change_registrar_, info->pref_name(),
- is_incognito_,
+ is_incognito_, store_last_modified,
base::Bind(&PrefProvider::Notify, base::Unretained(this)))));
}
@@ -155,6 +156,19 @@ bool PrefProvider::SetWebsiteSetting(
resource_identifier, in_value);
}
+base::Time PrefProvider::GetWebsiteSettingLastModified(
+ const ContentSettingsPattern& primary_pattern,
+ const ContentSettingsPattern& secondary_pattern,
+ ContentSettingsType content_type,
+ const ResourceIdentifier& resource_identifier) {
+ DCHECK(CalledOnValidThread());
+ DCHECK(prefs_);
+
+ return GetPref(content_type)
+ ->GetWebsiteSettingLastModified(primary_pattern, secondary_pattern,
+ resource_identifier);
+}
+
void PrefProvider::ClearAllContentSettingsRules(
ContentSettingsType content_type) {
DCHECK(CalledOnValidThread());

Powered by Google App Engine
This is Rietveld 408576698