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

Unified Diff: chrome/browser/resources/settings/reset_page/reset_profile_banner.js

Issue 2375223002: md-settings: Fix back navigation from /resetProfileSettings. (Closed)
Patch Set: Fix failing test. 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/reset_page/reset_profile_banner.js
diff --git a/chrome/browser/resources/settings/reset_page/reset_profile_banner.js b/chrome/browser/resources/settings/reset_page/reset_profile_banner.js
index 4c5aa929d0848ced1a7ce76dd419f0abd7084800..4ce02ea192b98643e2757ea6d500d4bc1d1da988 100644
--- a/chrome/browser/resources/settings/reset_page/reset_profile_banner.js
+++ b/chrome/browser/resources/settings/reset_page/reset_profile_banner.js
@@ -24,18 +24,15 @@ Polymer({
},
/**
- * Creates and shows a <settings-reset-profile-dialog>.
+ * Shows a <settings-reset-profile-dialog>.
* @private
*/
- showDialog_: function(dialogName) {
+ showDialog_: function() {
this.showResetProfileDialog_ = true;
- this.async(function() {
- var dialog = this.$$('settings-reset-profile-dialog');
- dialog.open();
- }.bind(this));
},
- onResetDone_: function() {
+ /** @private */
+ onDialogClose_: function() {
this.showResetProfileDialog_ = false;
},
});

Powered by Google App Engine
This is Rietveld 408576698