Write last_modified date to Content Settings in the PrefProvider

chrome/browser/content_settings/content_settings_internal_extension_provider.cc

Index: chrome/browser/content_settings/content_settings_internal_extension_provider.cc
diff --git a/chrome/browser/content_settings/content_settings_internal_extension_provider.cc b/chrome/browser/content_settings/content_settings_internal_extension_provider.cc
index bd58d07a74e21307b9b981047334aed280c616fc..8d31179ae9bfdc4418be26ee8ca4433f7f10445d 100644
--- a/chrome/browser/content_settings/content_settings_internal_extension_provider.cc
+++ b/chrome/browser/content_settings/content_settings_internal_extension_provider.cc
@@ -212,7 +212,7 @@ void InternalExtensionProvider::SetContentSettingForExtensionAndResource(
                              resource);
     } else {
       value_map_.SetValue(primary_pattern, secondary_pattern,
-                          CONTENT_SETTINGS_TYPE_PLUGINS, resource,
+                          CONTENT_SETTINGS_TYPE_PLUGINS, resource, base::Time(),

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

nit: Extension settings are processed when the extension is installed or at
browser startup; so I agree that a timestamp such as base::Time::Now() would be
meaningless here. But we should probably document it somewhere.

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

I added a comment to show that this is done intentionally

                           new base::Value(setting));
     }
   }