MD Downloads: remove malware nodes from DOM if a download isn't malware

chrome/browser/resources/md_downloads/item.html

 <link rel="import" href="chrome://resources/html/action_link.html">
 <link rel="import" href="chrome://resources/html/cr.html">
 <link rel="import" href="chrome://resources/html/load_time_data.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.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-material/paper-material.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-progress/paper-progress.html">
 <link rel="import" href="chrome://downloads/action_service.html">
 <link rel="import" href="chrome://downloads/constants.html">
 <link rel="import" href="chrome://downloads/throttled_icon_loader.html">
@@ skipping 48 matching lines @@
                 on-click="onResumeClick_">[[i18n_.resume]]</paper-button>
           </template>
           <template is="dom-if" if="[[showCancel_]]">
             <paper-button id="cancel"
                 on-click="onCancelClick_">[[i18n_.cancel]]</paper-button>
           </template>
           <span id="controlled-by"
               i18n-values=".innerHTML:controlByExtension"></span>
         </div>
 
         <div id="dangerous" class="controls" hidden$="[[!isDangerous_]]">

[tommycli, 2015/09/08 22:50:08]

Why not dom-if the outer layer for !isDangerous_?

[Dan Beam, 2015/09/08 23:08:08]

I will soon

           <!-- Dangerous file types (e.g. .exe, .jar). -->
-          <paper-button id="discard" i18n-content="dangerDiscard"
-              on-click="onDiscardDangerous_" class="discard"
-              hidden$="[[isMalware_]]"></paper-button>
-          <paper-button id="save" i18n-content="dangerSave"
-              on-click="onSaveDangerous_" class="keep"
-              hidden$="[[isMalware_]]"></paper-button>
+          <template is="dom-if" if="[[!isMalware_]]">
+            <paper-button id="discard" on-click="onDiscardDangerous_"
+                class="discard">[[i18n_.discard]]</paper-button>
+            <paper-button id="save" on-click="onSaveDangerous_"
+                class="keep">[[i18n_.save]]</paper-button>
+          </template>
 
           <!-- Things that safe browsing has determined to be dangerous. -->
-          <paper-button id="danger-remove" i18n-content="controlRemoveFromList"
-              on-click="onDiscardDangerous_" class="discard"
-              hidden$="[[!isMalware_]]"></paper-button>
-          <paper-button id="restore" i18n-content="dangerRestore"
-              on-click="onSaveDangerous_" class="keep"
-              hidden$="[[!isMalware_]]"></paper-button>
+          <template is="dom-if" if="[[isMalware_]]">
+            <paper-button id="danger-remove" on-click="onDiscardDangerous_"
+                class="discard">[[i18n_.remove]]</paper-button>
+            <paper-button id="restore" on-click="onSaveDangerous_"
+                class="keep">[[i18n_.restore]</paper-button>
+          </template>
         </div>
       </div>
 
       <div id="remove-wrapper" class="icon-wrapper">
         <paper-icon-button id="remove" icon="clear"
             on-click="onRemoveClick_"></paper-icon-button>
       </div>
 
       <div id="incognito" hidden$="[[!data_.otr]]"></div>
     </paper-material>
 
     <div id="end-cap"></div>
 
   </template>
   <link rel="import" type="css" href="chrome://resources/css/action_link.css">
   <link rel="import" type="css" href="chrome://downloads/shared_style.css">
   <link rel="import" type="css" href="chrome://downloads/item.css">
   <script src="chrome://downloads/item.js"></script>
 </dom-module>