MD Settings: Intenet: Fix network list

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

 <link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-collapse/iron-collapse.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-spinner/paper-spinner.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/paper-toggle-button.html">
 <link rel="import" href="chrome://resources/cr_elements/cr_expand_button/cr_expand_button.html">
 <link rel="import" href="chrome://resources/cr_elements/network/cr_network_list.html">
 <link rel="import" href="chrome://resources/cr_elements/network/cr_network_list_item.html">
 <link rel="import" href="/settings_shared_css.html">
 
 <dom-module name="network-summary-item">
   <template>
     <style include="settings-shared">
       paper-spinner {
         height: 20px;
         margin: 0 20px;
         width: 20px;
       }
 
       .button-row {
         align-items: center;
-        border-top: var(--settings-separator-line);
         display: flex;
         min-height: var(--settings-row-min-height);
       }
 
       #buttons {
         align-items: center;
       }
 
       #deviceEnabledButton {
         -webkit-margin-start: 10px;
@@ skipping 21 matching lines @@
           </template>
           <template is="dom-if" if="[[enableIsVisible_(deviceState)]]">
             <paper-toggle-button  id="deviceEnabledButton"
                 checked="[[deviceIsEnabled_(deviceState)]]"
                 on-tap="onDeviceEnabledTap_">
             </paper-toggle-button>
           </template>
         </div>
       </div>
     </div>
-    <template is="dom-if" if="[[expanded]]">
-      <div class="list-frame">
-        <cr-network-list id="networkList" max-height="[[maxHeight]]"
-            networks="[[networkStateList]]" opened="{{expanded}}" show-buttons>
-        </cr-network-list>
-        <template is="dom-if"
-            if="[[knownNetworksIsVisible_(activeNetworkState)]]">
-          <div class="button-row" actionable on-tap="onKnownNetworksTap_">
-            <a is="action-link">$i18n{knownNetworksButton}</a>
-          </div>
-        </template>
-      </div>
+    <template is="dom-if" if="[[domIfIsVisible_(expanded, wasExpanded)]]">
+      <iron-collapse opened="[[ironCollapseIsOpened_(expanded, wasExpanded)]]">

[dschuyler, 2016/08/18 23:30:18]

nit: The function names domIfIsVisible_ and 
ironCollapseIsOpened_ seem like they could
be improved. Maybe something about 
networkList instead of domIf and ironCollapse?

[stevenjb, 2016/08/19 00:20:43]

You mean like networkListDomIfIsVisible? That's getting pretty lengthy, but
sure...

[dschuyler, 2016/08/19 21:05:20]

This is just a suggestion, feel free to ignore it.
I thought domIfIsVisible was too generic. I agree
with not making it too long. Maybe 
isNetworkListVisible_

That would hold up better if a second dom-if were
added or if that dom-if were changed to hidden or
something else.

Maybe just isVisible_ would be ok too, but I prefer
isNetworkListVislbe_

Thanks for asking, so I could clarify what I meant. 
Whichever you decide on (even domIfIsVisible_, if ya
really feel that is best) is a LGTM.

[stevenjb, 2016/08/22 15:16:02]

I appreciate the suggestion, and after some though came up with
'networksDomIfIsTrue' (since dom-if is truthy, not shown/hidden). Not perfect,
but names are hard :)

+        <div class="list-frame">
+          <cr-network-list id="networkList" max-height="[[maxHeight]]"
+              networks="[[networkStateList]]" show-buttons>
+          </cr-network-list>
+          <template is="dom-if"
+              if="[[knownNetworksIsVisible_(activeNetworkState)]]">
+            <div class="button-row" actionable on-tap="onKnownNetworksTap_">
+              <a is="action-link">$i18n{knownNetworksButton}</a>
+            </div>
+          </template>
+        </div>
+      </iron-collapse>
     </template>
   </template>
   <script src="network_summary_item.js"></script>
 </dom-module>