Site Settings Desktop: Implement individual cookie removal and RemoveAll.

chrome/browser/resources/settings/site_settings/site_data.html

Index: chrome/browser/resources/settings/site_settings/site_data.html
diff --git a/chrome/browser/resources/settings/site_settings/site_data.html b/chrome/browser/resources/settings/site_settings/site_data.html
index 02540fd90fbe75853749f4e11ef2cc3154d83cc7..38f80c69ccbc2b41dedba01bedaddb2f3778175d 100644
--- a/chrome/browser/resources/settings/site_settings/site_data.html
+++ b/chrome/browser/resources/settings/site_settings/site_data.html
@@ -1,6 +1,7 @@
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout-classes.html">
+<link rel="import" href="/icons.html">
 <link rel="import" href="/settings_shared_css.html">
 <link rel="import" href="/site_settings/site_data_details_dialog.html">
 <link rel="import" href="/site_settings/site_settings_behavior.html">
@@ -15,24 +16,41 @@
       }
 
       .list-frame {
+        -webkit-margin-end: -29px;

[dschuyler, 2016/08/18 00:44:03]

Just curious: what is this fixing (or helping with)?

[Finnur, 2016/08/18 16:20:30]

Aligning the new trashcan so that it is under the other icons above the section.

         -webkit-padding-start: 36px;
         margin-top: 10px;
       }
 
+      .site-header {
+        margin-top: 10px;
+      }
+
       .site {
         margin-top: 3px;
       }
     </style>
-    <div>$i18n{siteSettingsCookieHeader}</div>
+    <div class="layout horizontal">
+      <div class="flex site-header">$i18n{siteSettingsCookieHeader}</div>
+      <div class="secondary-action" hidden$="[[!removeAllIsVisible_(sites)]]">
+        <paper-button on-tap="onDeleteAllSites_" class="secondary-button">
+          $i18n{siteSettingsCookieRemoveAll}
+        </paper-button>
+      </div>
+    </div>
+
     <div class="list-frame vertical-list">
       <template is="dom-repeat" items="[[sites]]">
-        <div class="list-item" on-tap="onSiteTap_" actionable>
+        <div class="list-item">
           <div class="favicon-image" style$="[[computeSiteIcon(item.site)]]"

[dschuyler, 2016/08/18 00:44:03]

The whole row area (other than the delete
button at the end) should be clickable to 
do the onSiteTap_. This may need an extra
div wrapping the icon and the item.* text.
That new div would have the on-tap="onSiteTap_"
and the actionable.

[Finnur, 2016/08/18 16:20:30]

Done.

-              on-tap="onOriginTap_"></div>
-          <div class="middle layout horizontal">
+              on-tap="onSiteTap_" actionable></div>
+          <div class="middle layout horizontal" on-tap="onSiteTap_" actionable>
             <div class="site flex">[[item.site]]</div>
             <div class="secondary">[[item.localData]]</div>
           </div>
+          <div>
+            <paper-icon-button on-tap="onDeleteSite_"
+                icon="settings:delete"></paper-icon-button>
+          </div>
         </div>
       </template>
     </div>