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

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

Issue 1698093007: Convert SiteSettingsCategory to use the HostContentSettingsMap instead of (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address feedback Created 4 years, 10 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: 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 3c9c2f4e7452b39b409d1b6fe1804d97033ed269..7ffd765aeb05e99ad336f75c566489babca4fd9d 100644
--- a/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
+++ b/chrome/browser/resources/settings/site_settings/site_settings_behavior.js
@@ -17,22 +17,6 @@ var SiteSettingsBehaviorImpl = {
},
/**
- * Returns whether the category default is set to enabled or not.
- * @param {number} category The category to check.
- * @return {boolean} True if the category default is set to enabled.
- * @protected
- */
- isCategoryAllowed: function(category) {
- var pref = this.getPref(this.computeCategoryPrefName(category));
-
- // FullScreen is Allow vs. Ask.
- if (category == settings.ContentSettingsTypes.FULLSCREEN)
- return pref.value != settings.PermissionValues.ASK;
-
- return pref.value != settings.PermissionValues.BLOCK;
- },
-
- /**
* Re-sets the category permission for a given origin.
* @param {string} origin The origin to change the permission for.
* @param {number} category The category permission to change.

Powered by Google App Engine
This is Rietveld 408576698