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

Unified Diff: chrome/browser/resources/settings/clear_browsing_data_page/clear_browsing_data_page.js

Issue 1750873002: [MD settings] change ClearBrowsingData from a sub-page to a dialog (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review changes Created 4 years, 10 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/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 5a9b14c2c61c0bdc9027ebcaaa5f5d433c7240e6..9c971a01f0af24469e2107384f2826aeb3942ab0 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
@@ -16,7 +16,7 @@
* </iron-animated-pages>
*
* @group Chrome Settings Elements
- * @element settings-privacy-page
+ * @element settings-clear-browsing-data-page
*/
Polymer({
is: 'settings-clear-browsing-data-page',
@@ -75,8 +75,7 @@ Polymer({
/** @private */
doneClearing_: function() {
- if (this.$)
- this.$.clearDataButton.disabled = false;
+ // TODO(dschuyler): inform the user on whether clearing data was successful.
},
/**
@@ -95,9 +94,15 @@ Polymer({
}
},
+ /** Open the clear browsing data dialog. */
+ showClearBrowsingDataDialog: function() {
+ this.$.clearBrowsingDataDialog.open();
+ },
+
/** @private */
- onPerformClearBrowsingDataTap_: function() {
- this.$.clearDataButton.disabled = true;
- chrome.send('performClearBrowserData');
+ onDialogClosed_: function(event) {
+ if (event.detail.confirmed) {
+ chrome.send('performClearBrowserData');
+ }
},
});

Powered by Google App Engine
This is Rietveld 408576698