Index: chrome/browser/resources/settings/site_settings/site_data_details_dialog.js |
diff --git a/chrome/browser/resources/settings/site_settings/site_data_details_dialog.js b/chrome/browser/resources/settings/site_settings/site_data_details_dialog.js |
index c9388e54651b9b272f606b3a194ce7b40e60d7f0..9af5239d8fbdb3a63788baa984b6aa06bb2e2b79 100644 |
--- a/chrome/browser/resources/settings/site_settings/site_data_details_dialog.js |
+++ b/chrome/browser/resources/settings/site_settings/site_data_details_dialog.js |
@@ -53,11 +53,12 @@ Polymer({ |
this.populateDialog_(); |
this.listener_ = cr.addWebUIListener( |
'onTreeItemRemoved', this.onTreeItemRemoved_.bind(this)); |
- this.$.dialog.open(); |
+ this.$.dialog.showModal(); |
}, |
/** |
* Populates the dialog with the data about the site. |
+ * @private |
*/ |
populateDialog_: function() { |
this.title_ = loadTimeData.getStringF('siteSettingsCookieDialog', |
@@ -84,6 +85,7 @@ Polymer({ |
* @param {string} currentPath The path constructed so far. |
* @param {string} targetId The id of the target leaf node to look for. |
* @return {string} The path of the node returned (or blank if not found). |
+ * @private |
*/ |
nodePath_: function(node, currentPath, targetId) { |
if (node.data_.id == targetId) |
@@ -104,6 +106,7 @@ Polymer({ |
* Add the cookie data to the content section of this dialog. |
* @param {string} id The id of the cookie node to display. |
* @param {!settings.CookieTreeNode} site The current site. |
+ * @private |
*/ |
populateItem_: function(id, site) { |
// Out with the old... |
@@ -118,6 +121,7 @@ Polymer({ |
site.addCookieData(root, node); |
}, |
+ /** @private */ |
onTreeItemRemoved_: function(args) { |
this.entries_ = this.site_.getCookieList(); |
if (args[0] == this.site_.data_.id || this.entries_.length == 0) { |
@@ -134,6 +138,7 @@ Polymer({ |
/** |
* A handler for when the user changes the dropdown box (switches cookies). |
+ * @private |
*/ |
onItemSelected_: function(event) { |
this.populateItem_(event.detail.selected, this.site_); |
@@ -150,6 +155,7 @@ Polymer({ |
/** |
* A handler for when the user opts to remove a single cookie. |
+ * @private |
*/ |
onRemove_: function(event) { |
this.browserProxy.removeCookie(this.nodePath_( |
@@ -158,9 +164,16 @@ Polymer({ |
/** |
* A handler for when the user opts to remove all cookies. |
+ * @private |
*/ |
onRemoveAll_: function(event) { |
cr.removeWebUIListener(this.listener_); |
this.browserProxy.removeCookie(this.site_.data_.id); |
+ this.$.dialog.close(); |
+ }, |
+ |
+ /** @private */ |
+ onCancelTap_: function() { |
+ this.$.dialog.cancel(); |
}, |
}); |