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

Side by Side Diff: ui/webui/resources/cr_elements/policy/cr_policy_pref_behavior.js

Issue 2708013003: [MD settings] show icon when content settings are controlled by an extension (Closed)
Patch Set: isPrefEnforced Created 3 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 unified diff | Download patch
« no previous file with comments | « ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview Behavior for policy controlled settings prefs. 6 * @fileoverview Behavior for policy controlled settings prefs.
7 */ 7 */
8 8
9 /** @polymerBehavior */ 9 /** @polymerBehavior */
10 var CrPolicyPrefBehavior = { 10 var CrPolicyPrefBehavior = {
11 /** 11 /**
12 * Is the |pref| controlled by something that prevents user control of the
13 * preference.
12 * @param {!chrome.settingsPrivate.PrefObject} pref 14 * @param {!chrome.settingsPrivate.PrefObject} pref
13 * @return {boolean} True if the pref is controlled by an enforced policy. 15 * @return {boolean} True if the pref is enforced.
14 */ 16 */
15 isPrefPolicyControlled: function(pref) { 17 isPrefEnforced: function(pref) {
16 return pref.enforcement == chrome.settingsPrivate.Enforcement.ENFORCED && 18 return pref.enforcement == chrome.settingsPrivate.Enforcement.ENFORCED;
17 pref.controlledBy != chrome.settingsPrivate.ControlledBy.EXTENSION;
18 }, 19 },
19 20
20 /** 21 /**
21 * @param {!chrome.settingsPrivate.ControlledBy|undefined} controlledBy 22 * @param {!chrome.settingsPrivate.ControlledBy|undefined} controlledBy
22 * @param {!chrome.settingsPrivate.Enforcement|undefined} enforcement 23 * @param {!chrome.settingsPrivate.Enforcement|undefined} enforcement
23 * @return {CrPolicyIndicatorType} The indicator type based on |controlledBy| 24 * @return {CrPolicyIndicatorType} The indicator type based on |controlledBy|
24 * and |enforcement|. 25 * and |enforcement|.
25 */ 26 */
26 getIndicatorType: function(controlledBy, enforcement) { 27 getIndicatorType: function(controlledBy, enforcement) {
27 if (enforcement == chrome.settingsPrivate.Enforcement.RECOMMENDED) 28 if (enforcement == chrome.settingsPrivate.Enforcement.RECOMMENDED)
28 return CrPolicyIndicatorType.RECOMMENDED; 29 return CrPolicyIndicatorType.RECOMMENDED;
29 if (enforcement == chrome.settingsPrivate.Enforcement.ENFORCED) { 30 if (enforcement == chrome.settingsPrivate.Enforcement.ENFORCED) {
30 switch (controlledBy) { 31 switch (controlledBy) {
31 case chrome.settingsPrivate.ControlledBy.PRIMARY_USER: 32 case chrome.settingsPrivate.ControlledBy.PRIMARY_USER:
32 return CrPolicyIndicatorType.PRIMARY_USER; 33 return CrPolicyIndicatorType.PRIMARY_USER;
33 case chrome.settingsPrivate.ControlledBy.OWNER: 34 case chrome.settingsPrivate.ControlledBy.OWNER:
34 return CrPolicyIndicatorType.OWNER; 35 return CrPolicyIndicatorType.OWNER;
35 case chrome.settingsPrivate.ControlledBy.USER_POLICY: 36 case chrome.settingsPrivate.ControlledBy.USER_POLICY:
36 return CrPolicyIndicatorType.USER_POLICY; 37 return CrPolicyIndicatorType.USER_POLICY;
37 case chrome.settingsPrivate.ControlledBy.DEVICE_POLICY: 38 case chrome.settingsPrivate.ControlledBy.DEVICE_POLICY:
38 return CrPolicyIndicatorType.DEVICE_POLICY; 39 return CrPolicyIndicatorType.DEVICE_POLICY;
40 case chrome.settingsPrivate.ControlledBy.EXTENSION:
41 return CrPolicyIndicatorType.EXTENSION;
Dan Beam 2017/02/25 00:13:41 please show me screenshots of situations like a <c
dschuyler 2017/02/28 00:15:18 Done off-line.
39 } 42 }
40 } 43 }
41 return CrPolicyIndicatorType.NONE; 44 return CrPolicyIndicatorType.NONE;
42 }, 45 },
43 }; 46 };
OLDNEW
« no previous file with comments | « ui/webui/resources/cr_elements/policy/cr_policy_indicator_behavior.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698