Write last_modified date to Content Settings in the PrefProvider

components/content_settings/core/browser/content_settings_pref_provider.cc

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..270ce81e44cbef7e09bc96ac06627a2f0e263726 100644
--- a/components/content_settings/core/browser/content_settings_pref_provider.cc
+++ b/components/content_settings/core/browser/content_settings_pref_provider.cc
@@ -81,8 +81,12 @@ void PrefProvider::RegisterProfilePrefs(
 }
 
 PrefProvider::PrefProvider(PrefService* prefs, bool incognito)
-    : prefs_(prefs),
-      is_incognito_(incognito) {
+    : PrefProvider(prefs, incognito, false) {}

[msramek, 2017/04/19 10:49:16]

Do we need this? HCSM needs to specify where to use the timestamps, and in
tests, I would rather be explicit which one we use. If you still prefer a
shortcut, you can just define in the test file.

[dullweber, 2017/04/19 15:02:45]

Done.

+
+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 +109,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)))));
   }