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

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

Issue 2387173002: [MD Settings][People] Disables profile name field for legacy supervised users. (Closed)
Patch Set: Addressed comments 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/people_page/manage_profile.js
diff --git a/chrome/browser/resources/settings/people_page/manage_profile.js b/chrome/browser/resources/settings/people_page/manage_profile.js
index ced1cf688ac133ef277693520441a3891c25ef78..e7f2901894ed870cb3fa312fb1167d3f6a3686e7 100644
--- a/chrome/browser/resources/settings/people_page/manage_profile.js
+++ b/chrome/browser/resources/settings/people_page/manage_profile.js
@@ -33,6 +33,12 @@ Polymer({
},
/**
+ * The current sync status.
+ * @type {?settings.SyncStatus}
+ */
+ syncStatus: Object,
+
+ /**
* @private {!settings.ManageProfileBrowserProxy}
*/
browserProxy_: {
@@ -75,4 +81,13 @@ Polymer({
this.browserProxy_.setProfileIconAndName(event.detail.selected,
this.profileName);
},
+
+ /**
+ * @param {?settings.SyncStatus} syncStatus
+ * @return {boolean} Whether the profile name field is disabled.
+ * @private
+ */
+ isProfileNameDisabled_: function(syncStatus) {
+ return !!syncStatus.supervisedUser && !syncStatus.childUser;
+ },
});

Powered by Google App Engine
This is Rietveld 408576698