Write last_modified date to Content Settings in the PrefProvider

chrome/browser/content_settings/content_settings_pref_provider_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/content_settings/core/browser/content_settings_pref_provider.h"
 
 #include <memory>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 578 matching lines @@
 
   for (const char* pref : nonempty_prefs) {
     DictionaryPrefUpdate update(&prefs, pref);
     const base::DictionaryValue* dictionary = update.Get();
     EXPECT_EQ(1u, dictionary->size());
   }
 
   provider.ShutdownOnUIThread();
 }
 
+TEST_F(PrefProviderTest, LastModified) {
+  sync_preferences::TestingPrefServiceSyncable prefs;
+  PrefProvider::RegisterProfilePrefs(prefs.registry());
+
+  ContentSettingsPattern pattern_1 =
+      ContentSettingsPattern::FromString("google.com");
+  ContentSettingsPattern pattern_2 =
+      ContentSettingsPattern::FromString("www.google.com");
+  std::unique_ptr<base::Value> value(new base::Value(CONTENT_SETTING_ALLOW));
+
+  base::Time t1 = base::Time::Now();

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

optional: This might be a good place for SimpleTestClock, but I'm personally
fine with the current coverage using LT and GT.

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

Thanks for pointing at this, it will probably be useful for other tests. I think
here it is easier to use timestamps than passing a clock from PrefProvider to
each ContentSettingsPref.

(At least if the clocks on all platforms are precise enough to ensure that
base::Time::Now() != base::Time::Now())

+  // Create a  provider and set a setting.
+  PrefProvider provider(&prefs, false, /*store_last_modified=*/true);
+  provider.SetWebsiteSetting(pattern_1, ContentSettingsPattern::Wildcard(),
+                             CONTENT_SETTINGS_TYPE_COOKIES, std::string(),
+                             value->DeepCopy());
+  provider.SetWebsiteSetting(pattern_2, ContentSettingsPattern::Wildcard(),
+                             CONTENT_SETTINGS_TYPE_COOKIES, std::string(),
+                             value->DeepCopy());
+  base::Time t2 = base::Time::Now();
+
+  {
+    std::unique_ptr<RuleIterator> it(provider.GetRuleIterator(
+        CONTENT_SETTINGS_TYPE_COOKIES, std::string(), false));
+    EXPECT_TRUE(it->HasNext());
+    Rule rule = it->Next();
+    EXPECT_EQ(pattern_1, rule.primary_pattern);
+    EXPECT_LT(t1, rule.last_modified);
+    EXPECT_GT(t2, rule.last_modified);
+
+    EXPECT_TRUE(it->HasNext());
+    rule = it->Next();
+    EXPECT_EQ(pattern_2, rule.primary_pattern);
+    EXPECT_LT(t1, rule.last_modified);
+    EXPECT_GT(t2, rule.last_modified);
+
+    EXPECT_FALSE(it->HasNext());
+  }
+
+  // Update pattern_1 and check that the timestamp changes.
+  std::unique_ptr<base::Value> value2(new base::Value(CONTENT_SETTING_BLOCK));
+  provider.SetWebsiteSetting(pattern_1, ContentSettingsPattern::Wildcard(),
+                             CONTENT_SETTINGS_TYPE_COOKIES, std::string(),
+                             value2->DeepCopy());
+  base::Time t3 = base::Time::Now();
+
+  {
+    std::unique_ptr<RuleIterator> it(provider.GetRuleIterator(
+        CONTENT_SETTINGS_TYPE_COOKIES, std::string(), false));
+    EXPECT_TRUE(it->HasNext());
+    Rule rule = it->Next();
+    EXPECT_EQ(pattern_1, rule.primary_pattern);
+    EXPECT_LT(t2, rule.last_modified);
+    EXPECT_GT(t3, rule.last_modified);
+
+    EXPECT_TRUE(it->HasNext());
+    rule = it->Next();
+    EXPECT_EQ(pattern_2, rule.primary_pattern);
+    EXPECT_LT(t1, rule.last_modified);
+    EXPECT_GT(t2, rule.last_modified);
+
+    EXPECT_FALSE(it->HasNext());
+  }
+
+  provider.ShutdownOnUIThread();
+}
+
 }  // namespace content_settings