MD Settings: Internet details: Fix proxy toggle and polish

chrome/browser/resources/settings/internet_page/network_proxy.html

Index: chrome/browser/resources/settings/internet_page/network_proxy.html
diff --git a/chrome/browser/resources/settings/internet_page/network_proxy.html b/chrome/browser/resources/settings/internet_page/network_proxy.html
index f2c2b8d900c21dd3c5daaeec3c29c3967251a923..3310d8e49307eca0050462af86828932ce484b4e 100644
--- a/chrome/browser/resources/settings/internet_page/network_proxy.html
+++ b/chrome/browser/resources/settings/internet_page/network_proxy.html
@@ -47,10 +47,6 @@
       #exceptionsDiv network-proxy-exclusions {
         margin: 10px 0;
       }
-
-      #proxyDiv paper-toggle-button {
-        padding: 10px 0;
-      }
     </style>
 
     <!-- Policy indicator. Only one dom-if below will be shown. -->
@@ -123,18 +119,20 @@
     </div>
 
     <!-- Manual -->
-    <div id="proxyDiv" class="settings-box continuation single-column indent"
+    <div class="settings-box continuation indent"
         hidden$="[[!matches_(proxy_.Type, ProxySettingsType_.MANUAL)]]">
-      <div class="flex layout horizontal">
-        <div id="networkProxyToggleLabel" class="flex">
-          $i18n{networkProxyUseSame}
-        </div>
-        <paper-toggle-button checked="{{useSameProxy_}}"
-            disabled="[[!isEditable_('Type', networkProperties, editable,
-                useSharedProxies_)]]"
-            aria-labelledby="networkProxyToggleLabel">
-        </paper-toggle-button>
+      <div id="networkProxyToggleLabel" class="flex">
+        $i18n{networkProxyUseSame}
       </div>
+      <paper-toggle-button checked="{{useSameProxy_}}"
+          disabled="[[!isEditable_('Type', networkProperties, editable,
+              useSharedProxies_)]]"
+          aria-labelledby="networkProxyToggleLabel">
+      </paper-toggle-button>
+    </div>
+
+    <div class="settings-box continuation single-column indent"
+        hidden$="[[!matches_(proxy_.Type, ProxySettingsType_.MANUAL)]]">

[Dan Beam, 2017/03/04 02:25:55]

every time i see a hidden$= instead of a dom-if, and angel loses its wings

[stevenjb, 2017/03/06 18:33:49]

If it helps you feel better, all of network-proxy is inside a dom-if :)

FWIW, I have a mental to-do to selectively replace hidden$ with dom-if
throughout Internet (which predates our use of dom-if), but will do so
separately (and it's not super high priority). I think we want to un-inline this
UI at the same time. I've introduced bugs by switching from hidden$ without
carefully auditing the JS (e.g. $. may not work, and properties need default
values).

       <div hidden$="[[!useSameProxy_]]" class="layout vertical">
         <network-proxy-input
             on-proxy-change="onProxyInputChange_"