MD Settings: Make PolicyControllable behavior for isPolicyControlled()

MD Settings: Make PolicyControllable behavior for isControlled()

This will let us add an isPolicyControlled() method to any
Polymer element with an associated pref.

BUG=521791
R=stevenjb@chromium.org,dbeam@chromium.org

Committed: https://crrev.com/3a91c55c3110879032376122b9f0e8f06efe54a4
Cr-Commit-Position: refs/heads/master@{#350105}

[michaelpg, 2015-09-20 21:07:18]

Our first behavior! WDYT?

[stevenjb, 2015-09-21 20:41:43]

Nice. lgtm w/ naming suggestion.

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/policy_controllable/policy_controllable.js
(right):

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/policy_controllable/policy_controllable.js:11:
isControlled: function(pref) {
nit: isPolicyControlled ?

[Dan Beam, 2015-09-21 22:27:11]

lgtm

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/policy_controllable/policy_controllable.js
(right):

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/policy_controllable/policy_controllable.js:11:
isControlled: function(pref) {
On 2015/09/21 20:41:43, stevenjb wrote:
> nit: isPolicyControlled ?

+1

[michaelpg, 2015-09-22 02:22:57]

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
File
chrome/browser/resources/settings/policy_controllable/policy_controllable.js
(right):

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/policy_controllable/policy_controllable.js:11:
isControlled: function(pref) {
On 2015/09/21 20:41:43, stevenjb wrote:
> nit: isPolicyControlled ?

Done.

https://codereview.chromium.org/1358433003/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/policy_controllable/policy_controllable.js:11:
isControlled: function(pref) {
On 2015/09/21 22:27:11, Dan Beam wrote:
> On 2015/09/21 20:41:43, stevenjb wrote:
> > nit: isPolicyControlled ?
> 
> +1

Acknowledged.

[michaelpg, 2015-09-22 02:23:01]

The CQ bit was checked by michaelpg@chromium.org

[michaelpg, 2015-09-22 02:23:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org, dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1358433003/#ps20001
(title: "isPolicyControlled()")

[commit-bot: I haz the power, 2015-09-22 02:23:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358433003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358433003/20001

[commit-bot: I haz the power, 2015-09-22 02:34:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-22 02:34:14]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[michaelpg, 2015-09-22 02:41:06]

The CQ bit was checked by michaelpg@chromium.org

[michaelpg, 2015-09-22 02:41:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org, dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/1358433003/#ps40001
(title: "i have brought great shame upon us")

[commit-bot: I haz the power, 2015-09-22 02:41:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358433003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358433003/40001

[commit-bot: I haz the power, 2015-09-22 03:43:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-22 03:43:15]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[michaelpg, 2015-09-22 03:45:57]

The CQ bit was checked by michaelpg@chromium.org

[commit-bot: I haz the power, 2015-09-22 03:46:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1358433003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1358433003/40001

[commit-bot: I haz the power, 2015-09-22 04:16:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-22 04:16:51]

Patchset 3 (id:??) landed as
https://crrev.com/3a91c55c3110879032376122b9f0e8f06efe54a4
Cr-Commit-Position: refs/heads/master@{#350105}