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

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

Issue 2853983002: Ensure settings returned from Content Settings providers are valid (Closed)
Patch Set: Ensure settings returned from Content Settings providers are valid Created 3 years, 8 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_registry_unittest.cc
diff --git a/components/content_settings/core/browser/content_settings_registry_unittest.cc b/components/content_settings/core/browser/content_settings_registry_unittest.cc
index d63d180e0bf8753d0c50ea957cab1257a2119f56..db419b6e434ae6974f59c0f7ea03f7e6266ff111 100644
--- a/components/content_settings/core/browser/content_settings_registry_unittest.cc
+++ b/components/content_settings/core/browser/content_settings_registry_unittest.cc
@@ -124,4 +124,23 @@ TEST_F(ContentSettingsRegistryTest, Iteration) {
EXPECT_TRUE(cookies_found);
}
+TEST_F(ContentSettingsRegistryTest, IsDefaultSettingValid) {
+ const ContentSettingsInfo* info =
+ registry()->Get(CONTENT_SETTINGS_TYPE_COOKIES);
+ EXPECT_TRUE(info->IsDefaultSettingValid(CONTENT_SETTING_ALLOW));
+
+#if !defined(OS_IOS)
+ info = registry()->Get(CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
+ EXPECT_FALSE(info->IsDefaultSettingValid(CONTENT_SETTING_ALLOW));
+
+ info = registry()->Get(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
+ EXPECT_FALSE(info->IsDefaultSettingValid(CONTENT_SETTING_ALLOW));
+#endif
+
+#if defined(OS_ANDROID) || defined(OS_CHROMEOS)
+ info = registry()->Get(CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER);
+ EXPECT_FALSE(info->IsDefaultSettingValid(CONTENT_SETTING_ALLOW));
+#endif
+}
+
} // namespace content_settings

Powered by Google App Engine
This is Rietveld 408576698