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

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

Issue 2554403005: [MD settings] move extra options in category settings (Closed)
Patch Set: merge with master; review changes Created 4 years 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_category.js
diff --git a/chrome/browser/resources/settings/site_settings/site_settings_category.js b/chrome/browser/resources/settings/site_settings/site_settings_category.js
deleted file mode 100644
index e065ca0aed80c18c0f298ad8404debe5aa9bd45b..0000000000000000000000000000000000000000
--- a/chrome/browser/resources/settings/site_settings/site_settings_category.js
+++ /dev/null
@@ -1,177 +0,0 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-/**
- * @fileoverview
- * 'site-settings-category' is the polymer element for showing a certain
- * category under Site Settings.
- */
-Polymer({
- is: 'site-settings-category',
-
- behaviors: [SiteSettingsBehavior, WebUIListenerBehavior],
-
- properties: {
- /**
- * Represents the state of the main toggle shown for the category. For
- * example, the Location category can be set to Block/Ask so false, in that
- * case, represents Block and true represents Ask.
- */
- categoryEnabled: Boolean,
-
- /**
- * The description to be shown next to the slider.
- * @private
- */
- sliderDescription_: String,
-
- /**
- * Used only for the Flash to persist the Ask First checkbox state.
- * Defaults to true, as the checkbox should be checked unless the user
- * has explicitly unchecked it or has the ALLOW setting on Flash.
- */
- flashAskFirst_: {
- type: Boolean,
- value: true,
- },
-
- /**
- * Used only for Cookies to keep track of the Session Only state.
- * Defaults to true, as the checkbox should be checked unless the user
- * has explicitly unchecked it or has the ALLOW setting on Cookies.
- */
- cookiesSessionOnly_: {
- type: Boolean,
- value: true,
- },
- },
-
- observers: [
- 'onCategoryChanged_(category)',
- ],
-
- ready: function() {
- this.addWebUIListener('contentSettingCategoryChanged',
- this.defaultValueForCategoryChanged_.bind(this));
- },
-
- /**
- * Called when the default value for a category has been changed.
- * @param {number} category The category that changed.
- * @private
- */
- defaultValueForCategoryChanged_: function(category) {
- if (category == this.category)
- this.onCategoryChanged_();
- },
-
- /**
- * A handler for changing the default permission value for a content type.
- * @private
- */
- onChangePermissionControl_: function() {
- switch (this.category) {
- case settings.ContentSettingsTypes.BACKGROUND_SYNC:
- case settings.ContentSettingsTypes.IMAGES:
- case settings.ContentSettingsTypes.JAVASCRIPT:
- case settings.ContentSettingsTypes.KEYGEN:
- case settings.ContentSettingsTypes.POPUPS:
- case settings.ContentSettingsTypes.PROTOCOL_HANDLERS:
- // "Allowed" vs "Blocked".
- this.browserProxy.setDefaultValueForContentType(
- this.category,
- this.categoryEnabled ?
- settings.PermissionValues.ALLOW :
- settings.PermissionValues.BLOCK);
- break;
- case settings.ContentSettingsTypes.AUTOMATIC_DOWNLOADS:
- case settings.ContentSettingsTypes.CAMERA:
- case settings.ContentSettingsTypes.GEOLOCATION:
- case settings.ContentSettingsTypes.MIC:
- case settings.ContentSettingsTypes.NOTIFICATIONS:
- case settings.ContentSettingsTypes.UNSANDBOXED_PLUGINS:
- // "Ask" vs "Blocked".
- this.browserProxy.setDefaultValueForContentType(
- this.category,
- this.categoryEnabled ?
- settings.PermissionValues.ASK :
- settings.PermissionValues.BLOCK);
- break;
- case settings.ContentSettingsTypes.COOKIES:
- // This category is tri-state: "Allow", "Block", "Keep data until
- // browser quits".
- var value = settings.PermissionValues.BLOCK;
- if (this.categoryEnabled) {
- value = this.cookiesSessionOnly_ ?
- settings.PermissionValues.SESSION_ONLY :
- settings.PermissionValues.ALLOW;
- }
- this.browserProxy.setDefaultValueForContentType(this.category, value);
- break;
- case settings.ContentSettingsTypes.PLUGINS:
- // This category is tri-state: "Allow", "Block", "Ask before running".
- var value = settings.PermissionValues.BLOCK;
- if (this.categoryEnabled) {
- value = this.flashAskFirst_ ?
- settings.PermissionValues.IMPORTANT_CONTENT :
- settings.PermissionValues.ALLOW;
- }
- this.browserProxy.setDefaultValueForContentType(this.category, value);
- break;
- default:
- assertNotReached('Invalid category: ' + this.category);
- }
- },
-
- /**
- * Handles changes to the category pref and the |category| member variable.
- * @private
- */
- onCategoryChanged_: function() {
- settings.SiteSettingsPrefsBrowserProxyImpl.getInstance()
- .getDefaultValueForContentType(
- this.category).then(function(defaultValue) {
- var setting = defaultValue.setting;
- this.categoryEnabled = this.computeIsSettingEnabled(setting);
-
- // Flash only shows ALLOW or BLOCK descriptions on the slider.
- var sliderSetting = setting;
- if (this.category == settings.ContentSettingsTypes.PLUGINS &&
- setting == settings.PermissionValues.IMPORTANT_CONTENT) {
- sliderSetting = settings.PermissionValues.ALLOW;
- } else if (
- this.category == settings.ContentSettingsTypes.COOKIES &&
- setting == settings.PermissionValues.SESSION_ONLY) {
- sliderSetting = settings.PermissionValues.ALLOW;
- }
- this.sliderDescription_ =
- this.computeCategoryDesc(this.category, sliderSetting, true);
-
- if (this.category == settings.ContentSettingsTypes.PLUGINS) {
- // The checkbox should only be cleared when the Flash setting
- // is explicitly set to ALLOW.
- if (setting == settings.PermissionValues.ALLOW)
- this.flashAskFirst_ = false;
- if (setting == settings.PermissionValues.IMPORTANT_CONTENT)
- this.flashAskFirst_ = true;
- } else if (
- this.category == settings.ContentSettingsTypes.COOKIES) {
- if (setting == settings.PermissionValues.ALLOW)
- this.cookiesSessionOnly_ = false;
- else if (setting == settings.PermissionValues.SESSION_ONLY)
- this.cookiesSessionOnly_ = true;
- }
- }.bind(this));
- },
-
- /** @private */
- isFlashCategory_: function(category) {
- return category == settings.ContentSettingsTypes.PLUGINS;
- },
-
- /** @private */
- isCookiesCategory_: function(category) {
- return category == settings.ContentSettingsTypes.COOKIES;
- },
-});

Powered by Google App Engine
This is Rietveld 408576698