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

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

Issue 1372353004: Making structure for ContentSettings and its corresponding strings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adding Test Case. Created 5 years, 2 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/content_settings_utils_unittest.cc
diff --git a/components/content_settings/core/browser/content_settings_utils_unittest.cc b/components/content_settings/core/browser/content_settings_utils_unittest.cc
index f89b45b42f9f4ad3eda913c973a263da655d0eca..c310d3d218dd5accaf0ab6cb1ffeccc3efe88edc 100644
--- a/components/content_settings/core/browser/content_settings_utils_unittest.cc
+++ b/components/content_settings/core/browser/content_settings_utils_unittest.cc
@@ -44,3 +44,31 @@ TEST(ContentSettingsUtilsTest, ParsePatternString) {
EXPECT_FALSE(pattern_pair.first.IsValid());
EXPECT_FALSE(pattern_pair.second.IsValid());
}
+
+TEST(ContentSettingsUtilsTest, ContentSettingsStringMap) {
+ // Checking content_settings::ContentSettingToString() functionality.
+ EXPECT_EQ("default",
+ content_settings::ContentSettingToString(CONTENT_SETTING_DEFAULT));
+ EXPECT_EQ("allow",
+ content_settings::ContentSettingToString(CONTENT_SETTING_ALLOW));
+ EXPECT_EQ("block",
+ content_settings::ContentSettingToString(CONTENT_SETTING_BLOCK));
+ EXPECT_EQ("ask",
+ content_settings::ContentSettingToString(CONTENT_SETTING_ASK));
+ EXPECT_EQ("session", content_settings::ContentSettingToString(
+ CONTENT_SETTING_SESSION_ONLY));
+ EXPECT_EQ("detect", content_settings::ContentSettingToString(
+ CONTENT_SETTING_DETECT_IMPORTANT_CONTENT));
+
+ // Checking content_settings::ContentSettingFromString() functionality.
+ EXPECT_EQ(CONTENT_SETTING_ALLOW,
+ content_settings::ContentSettingFromString("allow"));
+ EXPECT_EQ(CONTENT_SETTING_BLOCK,
+ content_settings::ContentSettingFromString("block"));
+ EXPECT_EQ(CONTENT_SETTING_ASK,
+ content_settings::ContentSettingFromString("ask"));
+ EXPECT_EQ(CONTENT_SETTING_SESSION_ONLY,
+ content_settings::ContentSettingFromString("session"));
+ EXPECT_EQ(CONTENT_SETTING_DETECT_IMPORTANT_CONTENT,
+ content_settings::ContentSettingFromString("detect"));
+}

Powered by Google App Engine
This is Rietveld 408576698