Chromium Code Reviews| Index: chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js |
| diff --git a/chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js b/chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js |
| index d5c56c6c8b00bc046fe07d30ec5890f54dc54aa8..a560e389f9b75e7ed118d8c1fc8659d062213777 100644 |
| --- a/chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js |
| +++ b/chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js |
| @@ -72,8 +72,7 @@ Polymer({ |
| /** @private */ |
| doneClearing_: function() { |
| - if (this.$) |
| - this.$.clearDataButton.disabled = false; |
| + // TODO(dschuyler): inform the user on whether clearing data was successful. |
| }, |
| /** |
| @@ -92,9 +91,15 @@ Polymer({ |
| } |
| }, |
| + /** Open the clear browsing data dialog. */ |
| + showClearBrowsingDataDialog: function() { |
|
dpapad
2016/03/09 20:13:24
If you end up renaming this Polymer element per pr
dschuyler
2016/03/09 22:15:16
Sounds good.
|
| + this.$.clearBrowsingDataDialog.open(); |
| + }, |
| + |
| /** @private */ |
| - onPerformClearBrowsingDataTap_: function() { |
| - this.$.clearDataButton.disabled = true; |
| - chrome.send('performClearBrowserData'); |
| + onDialogClosed_: function(event) { |
| + if (event.detail.confirmed) { |
| + chrome.send('performClearBrowserData'); |
| + } |
| }, |
| }); |