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

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

Issue 2102803002: Settings People Revamp: Add special Disconnect logic for domain profiles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address test feedback Created 4 years, 6 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/people_page/people_page.js
diff --git a/chrome/browser/resources/settings/people_page/people_page.js b/chrome/browser/resources/settings/people_page/people_page.js
index 668c7e630dac99fea0ead7e95975c4f40cb8feec..3053b544db8fbbe139388c2a316172fff82f8132 100644
--- a/chrome/browser/resources/settings/people_page/people_page.js
+++ b/chrome/browser/resources/settings/people_page/people_page.js
@@ -214,11 +214,17 @@ Polymer({
},
/** @private */
+ onDisconnectCancel_: function() {
+ this.$.disconnectDialog.close();
+ },
+
+ /** @private */
onDisconnectConfirm_: function() {
- var deleteProfile = this.$.deleteProfile && this.$.deleteProfile.checked;
+ var deleteProfile = !!this.syncStatus.domain ||
+ (this.$.deleteProfile && this.$.deleteProfile.checked);
this.syncBrowserProxy_.signOut(deleteProfile);
- // Dialog automatically closed because button has dialog-confirm attribute.
+ this.$.disconnectDialog.close();
},
/** @private */
@@ -274,6 +280,22 @@ Polymer({
/**
* @private
+ * @param {string} domain
+ * @return {string}
+ */
+ getDisconnectExplanationHtml_: function(domain) {
+<if expr="not chromeos">
+ if (domain) {
+ return loadTimeData.getStringF(
+ 'syncDisconnectManagedProfileExplanation',
+ '<span id="managed-by-domain-name">' + domain + '</span>');
+ }
+</if>
+ return loadTimeData.getString('syncDisconnectExplanation');
+ },
+
+ /**
+ * @private
* @param {?settings.SyncStatus} syncStatus
* @return {boolean}
*/

Powered by Google App Engine
This is Rietveld 408576698