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

Unified Diff: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm

Issue 2419413002: Deleted CONTENT_SETTINGS_TYPE_FULLSCREEN and MOUSELOCK. (Closed)
Patch Set: Fix more things. Created 4 years, 1 month 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/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm
diff --git a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm
index 0d18821cbff5d6ead8078c1fa3cd0289880b477d..3717ac2f8f80653994019fa3a569bb6d347cc33f 100644
--- a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm
+++ b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller_unittest.mm
@@ -62,33 +62,26 @@ enum PermissionMenuIndices {
};
const ContentSettingsType kTestPermissionTypes[] = {
- // NOTE: FULLSCREEN does not support "Always block", so it must appear as
- // one of the first three permissions.
- CONTENT_SETTINGS_TYPE_FULLSCREEN,
CONTENT_SETTINGS_TYPE_IMAGES,
CONTENT_SETTINGS_TYPE_JAVASCRIPT,
CONTENT_SETTINGS_TYPE_PLUGINS,
CONTENT_SETTINGS_TYPE_POPUPS,
CONTENT_SETTINGS_TYPE_GEOLOCATION,
CONTENT_SETTINGS_TYPE_NOTIFICATIONS,
- CONTENT_SETTINGS_TYPE_MOUSELOCK,
CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC
};
const ContentSetting kTestSettings[] = {
CONTENT_SETTING_DEFAULT,
CONTENT_SETTING_DEFAULT,
- CONTENT_SETTING_DEFAULT,
CONTENT_SETTING_ALLOW,
CONTENT_SETTING_BLOCK,
CONTENT_SETTING_ALLOW,
CONTENT_SETTING_BLOCK,
- CONTENT_SETTING_ALLOW,
CONTENT_SETTING_BLOCK
};
const ContentSetting kTestDefaultSettings[] = {
- CONTENT_SETTING_ALLOW,
CONTENT_SETTING_BLOCK,
CONTENT_SETTING_ASK
};
@@ -98,10 +91,8 @@ const content_settings::SettingSource kTestSettingSources[] = {
content_settings::SETTING_SOURCE_USER,
content_settings::SETTING_SOURCE_USER,
content_settings::SETTING_SOURCE_USER,
- content_settings::SETTING_SOURCE_USER,
content_settings::SETTING_SOURCE_POLICY,
content_settings::SETTING_SOURCE_POLICY,
- content_settings::SETTING_SOURCE_EXTENSION,
content_settings::SETTING_SOURCE_EXTENSION
};
@@ -288,9 +279,9 @@ TEST_F(WebsiteSettingsBubbleControllerTest, SetPermissionInfo) {
}
EXPECT_EQ(arraysize(kTestPermissionTypes), [labels count]);
- // 4 of the buttons should be disabled -- the ones that have a setting source
+ // 3 of the buttons should be disabled -- the ones that have a setting source
// of SETTING_SOURCE_POLICY or SETTING_SOURCE_EXTENSION.
- EXPECT_EQ(4, disabled_count);
+ EXPECT_EQ(3, disabled_count);
}
TEST_F(WebsiteSettingsBubbleControllerTest, WindowWidth) {

Powered by Google App Engine
This is Rietveld 408576698