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

Unified Diff: chrome/browser/content_settings/content_settings_policy_provider_unittest.cc

Issue 7328018: Migrate default geolocation content setting to host content settings map. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: " Created 9 years, 5 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/content_settings/content_settings_policy_provider_unittest.cc
diff --git a/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc b/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
index c827f188d2149f47b19dd505bd6c4fc491a6a907..a5fa4b86008e322f4499c53a5b916f5151206dfb 100644
--- a/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
+++ b/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
@@ -58,6 +58,45 @@ TEST_F(PolicyDefaultProviderTest, DefaultValues) {
provider.ShutdownOnUIThread();
}
+TEST_F(PolicyDefaultProviderTest, DefaultGeolocationContentSetting) {
+ TestingProfile profile;
+ TestingPrefService* prefs = profile.GetTestingPrefService();
+ PolicyDefaultProvider provider(prefs);
+
+ // By default, policies should be off.
+ EXPECT_FALSE(
+ provider.DefaultSettingIsManaged(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+ EXPECT_EQ(CONTENT_SETTING_DEFAULT,
+ provider.ProvideDefaultSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+
+ //
Bernhard Bauer 2011/07/15 13:31:44 Nit: Why the empty comment?
markusheintz_ 2011/07/15 16:21:50 Sry. Removed
+ prefs->SetInteger(prefs::kGeolocationDefaultContentSetting,
+ CONTENT_SETTING_ALLOW);
+ EXPECT_FALSE(
+ provider.DefaultSettingIsManaged(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+ EXPECT_EQ(CONTENT_SETTING_DEFAULT,
+ provider.ProvideDefaultSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+
+ //
+ prefs->SetManagedPref(prefs::kGeolocationDefaultContentSetting,
+ Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
+ EXPECT_FALSE(
+ provider.DefaultSettingIsManaged(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+ EXPECT_EQ(CONTENT_SETTING_DEFAULT,
+ provider.ProvideDefaultSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+
+ // Change the managed value of the default geolocation setting
+ prefs->SetManagedPref(prefs::kManagedDefaultGeolocationSetting,
+ Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
+
+ EXPECT_TRUE(
+ provider.DefaultSettingIsManaged(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+ EXPECT_EQ(CONTENT_SETTING_BLOCK,
+ provider.ProvideDefaultSetting(CONTENT_SETTINGS_TYPE_GEOLOCATION));
+
+ provider.ShutdownOnUIThread();
+}
+
// When a default-content-setting is set to a managed setting a
// CONTENT_SETTINGS_CHANGED notification should be fired. The same should happen
// if the managed setting is removed.

Powered by Google App Engine
This is Rietveld 408576698