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

Unified Diff: components/content_settings/core/browser/website_settings_registry_unittest.cc

Issue 1252073002: Move pref names and default value into WebsiteSettingsInfo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@website-settings-registry-simple
Patch Set: Created 5 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: components/content_settings/core/browser/website_settings_registry_unittest.cc
diff --git a/components/content_settings/core/browser/website_settings_registry_unittest.cc b/components/content_settings/core/browser/website_settings_registry_unittest.cc
index 5f9da12f065500f2cb558e98f7aa3e2856c2159a..85fcf6a01b660830147e36ab72defb35842dac07 100644
--- a/components/content_settings/core/browser/website_settings_registry_unittest.cc
+++ b/components/content_settings/core/browser/website_settings_registry_unittest.cc
@@ -5,6 +5,7 @@
#include "base/logging.h"
#include "components/content_settings/core/browser/website_settings_info.h"
#include "components/content_settings/core/browser/website_settings_registry.h"
+#include "components/content_settings/core/common/content_settings.h"
#include "components/content_settings/core/common/content_settings_types.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -39,4 +40,22 @@ TEST_F(WebsiteSettingsRegistryTest, GetByName) {
EXPECT_EQ(registry()->Get(CONTENT_SETTINGS_TYPE_COOKIES), info);
}
+TEST_F(WebsiteSettingsRegistryTest, Properties) {
+ const WebsiteSettingsInfo* info =
+ registry()->Get(CONTENT_SETTINGS_TYPE_COOKIES);
+ ASSERT_TRUE(info);
+ EXPECT_EQ("profile.content_settings.exceptions.cookies", info->pref_name());
+ EXPECT_EQ("profile.default_content_setting_values.cookies",
+ info->default_value_pref_name());
+ EXPECT_EQ(CONTENT_SETTING_ALLOW, info->initial_default_value());
+
+ info = registry()->Get(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
+ ASSERT_TRUE(info);
+ EXPECT_EQ("profile.content_settings.exceptions.media_stream_camera",
+ info->pref_name());
+ EXPECT_EQ("profile.default_content_setting_values.media_stream_camera",
+ info->default_value_pref_name());
+ EXPECT_EQ(CONTENT_SETTING_ASK, info->initial_default_value());
+}
+
} // namespace content_settings

Powered by Google App Engine
This is Rietveld 408576698