Add cr_policy_indicator for settings controls

chrome/browser/resources/settings/checkbox/checkbox.js

 // 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
  * `cr-settings-checkbox` is a checkbox that controls a supplied preference.
  *
  * Example:
  *      <cr-settings-checkbox pref="{{prefs.settings.enableFoo}}"
  *          label="Enable foo setting." subLabel="(bar also)">
  *      </cr-settings-checkbox>
  *
  * @element cr-settings-checkbox
  */
 Polymer({
   is: 'cr-settings-checkbox',
 
   properties: {
     /**
      * The boolean preference object to control.
-     * @type {?PrefObject}
+     * @type {?chrome.settingsPrivate.PrefObject}
      */
     pref: {
       type: Object,
-      notify: true
+      notify: true,
+      value: null

[Dan Beam, 2015/08/28 00:20:22]

nit: we are not IE, feel free to add trailing , so the blame doesn't change for
the last line of an object every time a new property is added

[stevenjb, 2015/08/28 23:18:07]

I forget where it's OK and where it's not; I think maybe it's not OK in closure
typedef'd objects? Anyway, done throughot.

[michaelpg, 2015/09/14 05:14:51]

also not OK in JSON

     },
 
+    /** Whether the checkbox should represent the inverted value. */
     inverted: {
       type: Boolean,
       value: false
     },
 
+    /** Whether the checkbox is checked. */
     checked: {
       type: Boolean,
       value: false,
       observer: 'checkedChanged_'
     },
 
+    /** Set to disable the element. */
+    disabled: {
+      type: Boolean,
+      value: false,
+      observer: 'disabledChanged_'
+    },
+
+    /** The disabled state of the checkbox (may be set by policy). */
+    checkboxDisabled: {
+      type: Boolean,
+      value: false
+    },
+
+    /** Checkbox label. */
     label: {
       type: String,
       value: '',
     },
 
+    /** Additional sub-label for the checkbox. */
     subLabel: {
       type: String,
       value: '',
     },
   },
 
   observers: [
-    'prefValueChanged_(pref.value)'
+    'prefPropertyChanged_(pref.*)'
   ],
 
   /** @override */
   ready: function() {
     this.$.events.forward(this.$.checkbox, ['change']);
   },
 
-  /** @private */
-  prefValueChanged_: function(prefValue) {
-    // prefValue is initially undefined when Polymer initializes pref.
-    if (prefValue !== undefined) {
-      this.checked = this.getNewValue_(prefValue);
+  /**
+   * Polymer observer for pref.
+   * @param {?{path: string, value: *}} change
+   * @private
+   */
+  prefPropertyChanged_: function(change) {
+    if (change.path == 'pref.value')
+      this.checked = this.getNewValue_(change.value);
+    if (change.path == 'pref.policySource' ||

[michaelpg, 2015/08/27 23:40:01]

else if

[stevenjb, 2015/08/28 23:18:07]

Done.

+        change.path == 'pref.policyEnforcement') {
+      this.checkboxDisabled =
+          this.disabled || this.isSettingDisabled_(this.pref);
     }
   },
 
-  /** @private */
+  /**
+   * Polymer observer for checked.
+   * @private
+   */
   checkedChanged_: function() {
-    if (this.pref) {
-      this.pref.value = this.getNewValue_(this.checked);
-    }
+    if (!this.pref)
+      return;
+    this.pref.value = this.getNewValue_(this.checked);
   },
 
-  /** @private */
-  getNewValue_: function(val) {
-    return this.inverted ? !val : val;
-  }
+  /**
+   * Polymer observer for disabled.
+   * @private
+   */
+  disabledChanged_: function() {
+    this.checkboxDisabled = this.disabled || this.isSettingDisabled_(this.pref);

[michaelpg, 2015/08/27 23:40:01]

nit: instead of adding listeners like disabledChanged and pref.*, can you bind
the HTML "disabled" attribute to a function which checks pref,
pref.policyEnforcement and disabled

[stevenjb, 2015/08/28 23:18:07]

Probably... I'll look into that after I merge with you prefs.js CL.

[michaelpg, 2015/09/14 05:14:51]

could you revisit this? I think having a computed attribute would make the code
simpler.

[stevenjb, 2015/09/14 23:05:30]

Done.

+  },
+
+  /**
+   * @param {*} value
+   * @return {boolean} The value as a boolean, inverted if |inverted| is true.
+   * @private
+   */
+  getNewValue_: function(value) {
+    return this.inverted ? !value : !!value;
+  },
+
+  /**
+   * @param {?chrome.settingsPrivate.PrefObject} pref
+   * @return {boolean} True if the setting is either explicitly disabled, or
+   *     disabled by the policyEnforcement property.
+   */
+  isSettingDisabled_: function(pref) {
+    return !!pref && pref.policyEnforcement ==
+                         chrome.settingsPrivate.PolicyEnforcement.ENFORCED;
+  },
 });