Settings: Internet page cleanup

chrome/browser/resources/settings/settings_shared_css.html

 <link rel="import" href="chrome://resources/cr_elements/shared_style_css.html">
 <link rel="import" href="settings_vars_css.html">
 
 <!-- Common styles for Material Design settings. -->
 <dom-module id="settings-shared">
   <template>
     <style include="cr-shared-style">
       :host-context([dir=rtl]) button[is='paper-icon-button-light'] {
         transform: scaleX(-1);  /* Flip on the X axis (aka mirror). */
       }
@@ skipping 58 matching lines @@
       .primary-button {
         --paper-button: {
           -webkit-padding-end: var(--settings-button-edge-spacing);
           -webkit-padding-start: var(--settings-button-edge-spacing);
           font-weight: 500;
           text-align: start;
         };
         --paper-button-flat-keyboard-focus: {
           background: rgba(51, 103, 214, .12);  /* --google-blue-700 */
         };
+      }
+
+      .primary-button[disabled] {
+        background: none;
+      }
+
+      .primary-button:not([disabled]) {

[dschuyler, 2017/05/18 18:20:10]

Would it work out the same to leave this color in .primary-button and then
define

.primary-button[disabled] {
  background: none;
  color: var(--paper-grey-600);
}

That seems clearer to me. By not having to work out the negative (:not()).

[stevenjb, 2017/05/18 19:06:05]

Done.

         color: var(--google-blue-500);
       }
 
       paper-toggle-button {
         @apply(--settings-actionable);
         height: var(--settings-row-min-height);
         width: 36px;
       }
 
       span ~ a {
@@ skipping 302 matching lines @@
         transform: rotate(-135deg);
       }
 
       .column-header {
         color: var(--paper-grey-600);
         font-weight: 500;
       }
     </style>
   </template>
 </dom-module>