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

Unified Diff: chrome/test/data/webui/settings/site_settings_category_tests.js

Issue 2501763003: Remove fullscreen from MD Settings internal lists. (Closed)
Patch Set: Fixed tests. 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/test/data/webui/settings/site_settings_category_tests.js
diff --git a/chrome/test/data/webui/settings/site_settings_category_tests.js b/chrome/test/data/webui/settings/site_settings_category_tests.js
index aff8c4d9d76aef286ce5ac7459862e15fe87166f..205667da1268a4fd2d98c07be4ac7107ed49e00b 100644
--- a/chrome/test/data/webui/settings/site_settings_category_tests.js
+++ b/chrome/test/data/webui/settings/site_settings_category_tests.js
@@ -130,10 +130,6 @@ cr.define('site_settings_category', function() {
// A quick testing note on a few special categories...
- // Full Screen is not a top-level category, and only appears as a
- // permission under Site Details. Therefore it has no Category Desc
- // that can be tested for.
-
// The USB Devices category has no global toggle -- and therefore no
// Category Desc (and has a special way of storing its data).
@@ -143,8 +139,7 @@ cr.define('site_settings_category', function() {
// Test category text ids and descriptions for those categories that
// have those.
- if (category != settings.ContentSettingsTypes.FULLSCREEN &&
- category != settings.ContentSettingsTypes.USB_DEVICES &&
+ if (category != settings.ContentSettingsTypes.USB_DEVICES &&
category != settings.ContentSettingsTypes.ZOOM_LEVELS) {
assertNotEquals('', testElement.computeCategoryDesc(
category, settings.PermissionValues.ALLOW, true));

Powered by Google App Engine
This is Rietveld 408576698