MD Settings: Stop rendering unnecessary iron-collapse instances in cert manager.

chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html

Index: chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html
diff --git a/chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html b/chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html
index ee0f93a79cbf96c870bcaf5f17b2b9fe9433c391..31cb15c857e3ed0fd3661c32e829c96ccadc55ab 100644
--- a/chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html
+++ b/chrome/browser/resources/settings/certificate_manager_page/certificate_entry.html
@@ -14,8 +14,8 @@
           alt="$i18n{certificateManagerExpandA11yLabel}">
       </cr-expand-button>
     </div>
-    <iron-collapse opened="[[expanded_]]" no-animation>
-      <template is="dom-if" if="[[expanded_]]">
+    <template is="dom-if" if="[[expanded_]]">

[dpapad, 2017/04/05 00:36:39]

Moving the dom-if outside the iron-collapse saves about 100+ iron-collapse
elements which used to be rendered when the "Authorities" tab was clicked. Now
each one of them is renedred only if the user requests it.

+      <iron-collapse opened="[[expanded_]]" no-animation>

[dpapad, 2017/04/05 00:36:39]

We our using no-animation for our iron-collapse, which essentially makes it a
completely unnecessary wrapper around the list-frame div below. We should
probably remove this completely in a follow up CL.

         <div class="list-frame">
           <template is="dom-repeat" items="[[model.subnodes]]">
             <settings-certificate-subentry model="[[item]]"
@@ -24,8 +24,8 @@
             </settings-certificate-subentry>
           </template>
         </div>
-      </template>
-    <iron-collapse>
+      <iron-collapse>
+    </template>
   </template>
   <script src="certificate_entry.js"></script>
 </dom-module>