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

Unified Diff: chrome/browser/resources/settings/site_settings/site_settings_behavior.js

Issue 2419413002: Deleted CONTENT_SETTINGS_TYPE_FULLSCREEN and MOUSELOCK. (Closed)
Patch Set: Fix Mac. 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/resources/settings/site_settings/site_settings_behavior.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
index bd0b1ca0488bfda9edfe70df70cb118e0b123865..f467e8873057de00359308b28eefea21dfa5eedc 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
@@ -397,16 +397,12 @@ var SiteSettingsBehaviorImpl = {
/**
* Returns true if the passed content setting is considered 'enabled'.
- * @param {string} category
* @param {string} setting
* @return {boolean}
* @private
*/
- computeIsSettingEnabled: function(category, setting) {
- // FullScreen is Allow vs. Ask.
- return category == settings.ContentSettingsTypes.FULLSCREEN ?
- setting != settings.PermissionValues.ASK :
- setting != settings.PermissionValues.BLOCK;
+ computeIsSettingEnabled: function(setting) {
+ return setting != settings.PermissionValues.BLOCK;
},
/**

Powered by Google App Engine
This is Rietveld 408576698