[MD settings] content settings add site button; layout fixes

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

Index: chrome/browser/resources/settings/site_settings/site_data.js
diff --git a/chrome/browser/resources/settings/site_settings/site_data.js b/chrome/browser/resources/settings/site_settings/site_data.js
index 245561a0bb8cb18905ce32acde5f553550a35222..e17c67bf4926b9f00c2b0cb57090bbaf1a5c7bef 100644
--- a/chrome/browser/resources/settings/site_settings/site_data.js
+++ b/chrome/browser/resources/settings/site_settings/site_data.js
@@ -71,7 +71,7 @@ Polymer({
   /**
    * A filter function for the list.
    * @param {!CookieDataSummaryItem} item The item to possibly filter out.
-   * @return {!boolean} Whether to show the item.
+   * @return {boolean} Whether to show the item.
    * @private
    */
   showItem_: function(item) {
@@ -93,7 +93,7 @@ Polymer({
 
   /**
    * Returns the string to use for the Remove label.
-   * @return {!string} filterString The current filter string.
+   * @return {string} filterString The current filter string.
    * @private
    */
   computeRemoveLabel_: function(filterString) {
@@ -157,18 +157,6 @@ Polymer({
   },
 
   /**
-   * Shows a dialog to confirm the deletion of a site.
-   * @param {!{model: !{item: CookieDataSummaryItem}}} event
-   * @private
-   */
-  onConfirmDeleteSite_: function(event) {
-    this.idToDelete_ = event.model.item.id;
-    this.confirmationDeleteMsg_ = loadTimeData.getStringF(
-        'siteSettingsCookieRemoveConfirmation', event.model.item.site);

[dpapad, 2016/10/19 19:40:13]

Is the 'siteSettingsCookieRemoveConfirmation' not used anymore? Can we fully
remove if so?

[dschuyler, 2016/10/20 00:06:09]

done.

-    this.$.confirmDeleteDialog.showModal();
-  },
-
-  /**
    * Shows a dialog to confirm the deletion of multiple sites.
    * @private
    */