Chromium Code Reviews| Index: chrome/browser/resources/settings/reset_page/reset_profile_dialog.js |
| diff --git a/chrome/browser/resources/settings/reset_page/reset_profile_dialog.js b/chrome/browser/resources/settings/reset_page/reset_profile_dialog.js |
| index b82b347793f5245ba5a7e50a08dc8892360273b2..2798360cf2d810362cde3f221bde371be3720991 100644 |
| --- a/chrome/browser/resources/settings/reset_page/reset_profile_dialog.js |
| +++ b/chrome/browser/resources/settings/reset_page/reset_profile_dialog.js |
| @@ -4,8 +4,11 @@ |
| /** |
| * @fileoverview |
| + * |
| * 'settings-reset-profile-dialog' is the dialog shown for clearing profile |
| - * settings. |
| + * settings. A triggered variant of this dialog can be shown under certain |
| + * circumstances. See triggered_profile_resetter.h for when the triggered |
| + * variant will be used. |
| */ |
| Polymer({ |
| is: 'settings-reset-profile-dialog', |
| @@ -15,6 +18,19 @@ Polymer({ |
| properties: { |
| // TODO(dpapad): Evaluate whether this needs to be synced across different |
| // settings tabs. |
| + |
| + /** @private */ |
| + isTriggered_: { |
| + type: Boolean, |
| + value: false, |
| + }, |
| + |
| + /** @private */ |
| + triggeredResetToolName_: { |
| + type: String, |
| + value: '', |
| + }, |
| + |
| /** @private */ |
| clearingInProgress_: { |
| type: Boolean, |
| @@ -25,6 +41,30 @@ Polymer({ |
| /** @private {!settings.ResetBrowserProxy} */ |
| browserProxy_: null, |
| + /** |
| + * @private |
| + * @return {string} |
| + */ |
| + getExplanationText_: function() { |
| + if (this.isTriggered_) { |
| + return loadTimeData.getStringF('triggeredResetPageExplanation', |
| + this.triggeredResetToolName_); |
| + } |
| + return loadTimeData.getStringF('resetPageExplanation'); |
| + }, |
| + |
| + /** |
| + * @private |
| + * @return {string} |
| + */ |
| + getPageTitle_: function() { |
| + if (this.isTriggered_) { |
| + return loadTimeData.getStringF('triggeredResetPageTitle', |
| + this.triggeredResetToolName_); |
| + } |
| + return loadTimeData.getStringF('resetPageTitle'); |
| + }, |
| + |
| /** @override */ |
| ready: function() { |
| this.browserProxy_ = settings.ResetBrowserProxyImpl.getInstance(); |
| @@ -34,9 +74,19 @@ Polymer({ |
| }.bind(this)); |
| }, |
| - open: function() { |
| - this.$.dialog.showModal(); |
| - this.browserProxy_.onShowResetProfileDialog(); |
| + /** @param {boolean} isTriggered */ |
| + open: function(isTriggered) { |
| + this.isTriggered_ = isTriggered; |
| + if (isTriggered) { |
| + this.browserProxy_.getTriggeredResetToolName().then(function(name) { |
| + this.triggeredResetToolName_ = name; |
| + this.$.dialog.showModal(); |
| + this.browserProxy_.onShowResetProfileDialog(); |
| + }.bind(this)); |
| + } else { |
| + this.$.dialog.showModal(); |
| + this.browserProxy_.onShowResetProfileDialog(); |
|
Dan Beam
2016/09/28 06:12:19
maybe combine these 2 lines into a method?
showDi
alito
2016/10/11 19:02:49
Done.
|
| + } |
| }, |
| /** @private */ |