MD Settings: change most checkboxes to toggles (leave dialogs alone)

chrome/browser/resources/settings/controls/controlled_button.html

 <link rel="import" href="chrome://resources/html/assert.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/cr_elements/policy/cr_policy_pref_behavior.html">
 <link rel="import" href="chrome://resources/cr_elements/policy/cr_policy_pref_indicator.html">
+<link rel="import" href="chrome://resources/cr_elements/policy/cr_policy_indicator_behavior.html">
 <link rel="import" href="/controls/pref_control_behavior.html">
 <link rel="import" href="/settings_shared_css.html">
 
 <dom-module id="controlled-button">
   <template>
     <style include="settings-shared">
       :host {
         align-items: center;
         display: flex;
       }
 
       :host([controlled_]) {
         /* Disable pointer events for this whole element, as outer on-tap gets
          * triggered when clicking/tapping anywhere in :host. */
         pointer-events: none;
       }
 
       cr-policy-pref-indicator {
-        -webkit-margin-start: var(--checkbox-spacing);
         /* Enable pointer events for the indicator so :hover works. Disable
          * clicks/taps via onIndicatorTap_ so outer on-tap doesn't trigger. */
         pointer-events: all;
       }
+
+      :host(:not([right-justified])) cr-policy-pref-indicator {
+        -webkit-margin-start: var(--checkbox-spacing);

[dschuyler, 2017/01/05 20:25:05]

optional since this is just moving exiting code, but the name --checkbox-spacing
seems odd to apply to a policy pref indicator. Maybe there should be a new var,
or maybe that var should be renamed.

[Dan Beam, 2017/01/06 02:45:29]

Done.

+      }
+
+      :host([right-justified]) cr-policy-pref-indicator {
+        -webkit-margin-end: var(--checkbox-spacing);
+        order: -1;
+      }
     </style>
 
     <paper-button disabled="[[controlled_]]">
       <content></content>
     </paper-button>
 
-    <template is="dom-if" if="[[controlled_]]">
+    <template is="dom-if" if="[[showIndicator_(pref.controlledBy,
+                                               pref.enforcement)]]">

[dschuyler, 2017/01/05 20:25:05]

This seems different from the checkbox to toggle comment in the CL description?
I'm wondering if the CL description could be amended to include a comment about
these changes.

[Dan Beam, 2017/01/06 02:45:29]

split to a separate CL.

       <cr-policy-pref-indicator pref="[[pref]]" on-tap="onIndicatorTap_">
       </cr-policy-pref-indicator>
     </template>
 
   </template>
   <script src="/controls/controlled_button.js"></script>
 </dom-module>