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

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

Issue 1838213002: Simplify Site Settings tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address feedback Created 4 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
« no previous file with comments | « chrome/test/data/webui/settings/site_settings_category_tests.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
diff --git a/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js b/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
index a7b29821c43ba1ecefb91df77e8f98b6115a97b8..a05734d0a98b2425c85877057d05fb97c26b69d8 100644
--- a/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
+++ b/chrome/test/data/webui/settings/test_site_settings_prefs_browser_proxy.js
@@ -74,27 +74,6 @@ TestSiteSettingsPrefsBrowserProxy.prototype = {
/** @override */
setDefaultValueForContentType: function(contentType, defaultValue) {
- if (contentType == settings.ContentSettingsTypes.CAMERA)
- this.prefs_.defaults.media_stream_camera = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.COOKIES)
- this.prefs_.defaults.cookies = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.FULLSCREEN)
- this.prefs_.defaults.fullscreen = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.GEOLOCATION)
- this.prefs_.defaults.geolocation = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.IMAGES)
- this.prefs_.defaults.images = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.JAVASCRIPT)
- this.prefs_.defaults.javascript = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.MIC)
- this.prefs_.defaults.media_stream_mic = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.NOTIFICATIONS)
- this.prefs_.defaults.notifications = defaultValue == 'allow';
- else if (contentType == settings.ContentSettingsTypes.POPUPS)
- this.prefs_.defaults.popups = defaultValue == 'allow';
- else
- console.log('setDefault received unknown category: ' + contentType);
-
this.methodCalled(
'setDefaultValueForContentType', [contentType, defaultValue]);
},
« no previous file with comments | « chrome/test/data/webui/settings/site_settings_category_tests.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698