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

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, 4 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..f9ae40bc55535444a5177e0abc9dbc7bd9303dc7 100644
--- a/components/content_settings/core/browser/website_settings_registry_unittest.cc
+++ b/components/content_settings/core/browser/website_settings_registry_unittest.cc
@@ -3,8 +3,10 @@
// found in the LICENSE file.
#include "base/logging.h"
+#include "base/values.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 +41,25 @@ 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());
+ int setting;
+ ASSERT_TRUE(info->initial_default_value()->GetAsInteger(&setting));
+ EXPECT_EQ(CONTENT_SETTING_ALLOW, setting);
+
+ 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());
+ ASSERT_TRUE(info->initial_default_value()->GetAsInteger(&setting));
+ EXPECT_EQ(CONTENT_SETTING_ASK, setting);
+}
+
} // namespace content_settings

Powered by Google App Engine
This is Rietveld 408576698