Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(265)

Unified Diff: chrome/browser/resources/settings/site_settings/site_data.js

Issue 2429973003: [MD settings] content settings add site button; layout fixes (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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);
- this.$.confirmDeleteDialog.showModal();
- },
-
- /**
* Shows a dialog to confirm the deletion of multiple sites.
* @private
*/

Powered by Google App Engine
This is Rietveld 408576698