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

Unified Diff: chrome/browser/policy/configuration_policy_handler_unittest.cc

Issue 15199006: Retire ClearSiteDataOnExit policy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 7 years, 7 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: chrome/browser/policy/configuration_policy_handler_unittest.cc
diff --git a/chrome/browser/policy/configuration_policy_handler_unittest.cc b/chrome/browser/policy/configuration_policy_handler_unittest.cc
index 95e5256c690ee4dd3a686a5369cde3494a200dbd..a57f1fb489e875458e8e3e6167732ba2ecddf86b 100644
--- a/chrome/browser/policy/configuration_policy_handler_unittest.cc
+++ b/chrome/browser/policy/configuration_policy_handler_unittest.cc
@@ -686,57 +686,4 @@ TEST(ExtensionURLPatternListPolicyHandlerTest, ApplyPolicySettings) {
EXPECT_TRUE(base::Value::Equals(&list, value));
}
-TEST(ClearSiteDataOnExitPolicyHandlerTest, CheckPolicySettings) {
- ClearSiteDataOnExitPolicyHandler handler;
- PolicyMap policy_map;
- PolicyErrorMap errors;
-
- EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors));
- EXPECT_TRUE(errors.empty());
-
- policy_map.Set(key::kClearSiteDataOnExit, POLICY_LEVEL_MANDATORY,
- POLICY_SCOPE_USER, base::Value::CreateBooleanValue(true));
- errors.Clear();
- EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors));
- EXPECT_TRUE(errors.empty());
-
- policy_map.Set(key::kDefaultCookiesSetting, POLICY_LEVEL_MANDATORY,
- POLICY_SCOPE_USER,
- base::Value::CreateIntegerValue(CONTENT_SETTING_ALLOW));
- errors.Clear();
- EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors));
- EXPECT_FALSE(errors.empty());
- EXPECT_FALSE(errors.GetErrors(key::kDefaultCookiesSetting).empty());
-}
-
-TEST(ClearSiteDataOnExitPolicyHandlerTest, ApplyPolicySettings) {
- ClearSiteDataOnExitPolicyHandler handler;
- PolicyMap policy_map;
- PrefValueMap prefs;
- const base::Value* val = NULL;
-
- handler.ApplyPolicySettings(policy_map, &prefs);
- EXPECT_FALSE(prefs.GetValue(prefs::kManagedDefaultCookiesSetting, &val));
-
- policy_map.Set(key::kClearSiteDataOnExit, POLICY_LEVEL_MANDATORY,
- POLICY_SCOPE_USER, base::Value::CreateBooleanValue(true));
- prefs.Clear();
- handler.ApplyPolicySettings(policy_map, &prefs);
- ASSERT_TRUE(prefs.GetValue(prefs::kManagedDefaultCookiesSetting, &val));
- EXPECT_TRUE(base::FundamentalValue(CONTENT_SETTING_SESSION_ONLY).Equals(val));
-
- policy_map.Set(key::kDefaultCookiesSetting, POLICY_LEVEL_MANDATORY,
- POLICY_SCOPE_USER,
- base::Value::CreateIntegerValue(CONTENT_SETTING_ALLOW));
- prefs.Clear();
- handler.ApplyPolicySettings(policy_map, &prefs);
- ASSERT_TRUE(prefs.GetValue(prefs::kManagedDefaultCookiesSetting, &val));
- EXPECT_TRUE(base::FundamentalValue(CONTENT_SETTING_SESSION_ONLY).Equals(val));
-
- policy_map.Clear();
- prefs.Clear();
- handler.ApplyPolicySettings(policy_map, &prefs);
- EXPECT_FALSE(prefs.GetValue(prefs::kManagedDefaultCookiesSetting, &val));
-}
-
} // namespace policy
« no previous file with comments | « chrome/browser/policy/configuration_policy_handler_list.cc ('k') | chrome/browser/policy/policy_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698