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

Unified Diff: chrome/test/data/webui/settings/people_page_manage_profile_test.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
« no previous file with comments | « chrome/browser/resources/settings/people_page/people_page.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/people_page_manage_profile_test.js
diff --git a/chrome/test/data/webui/settings/people_page_manage_profile_test.js b/chrome/test/data/webui/settings/people_page_manage_profile_test.js
index 6d9fd3e2310ba13527820a26915d8a14f9930b1a..0c63889f5d0ab7660b8547018971f2916d97ee2f 100644
--- a/chrome/test/data/webui/settings/people_page_manage_profile_test.js
+++ b/chrome/test/data/webui/settings/people_page_manage_profile_test.js
@@ -43,6 +43,7 @@ cr.define('settings_people_page_manage_profile', function() {
manageProfile = document.createElement('settings-manage-profile');
manageProfile.profileIconUrl = 'fake-icon-1.png';
manageProfile.profileName = 'Initial Fake Name';
+ manageProfile.syncStatus = {supervisedUser: false, childUser: false};
document.body.appendChild(manageProfile);
});
@@ -93,6 +94,7 @@ cr.define('settings_people_page_manage_profile', function() {
test('ManageProfileChangeName', function() {
var nameField = manageProfile.$.name;
assertTrue(!!nameField);
+ assertFalse(!!nameField.disabled);
assertEquals('Initial Fake Name', nameField.value);
@@ -106,6 +108,16 @@ cr.define('settings_people_page_manage_profile', function() {
});
});
+ test('ProfileNameIsDisabledForSupervisedUser', function() {
+ manageProfile.syncStatus = {supervisedUser: true, childUser: false};
+
+ var nameField = manageProfile.$.name;
+ assertTrue(!!nameField);
+
+ // Name field should be disabled for legacy supervised users.
+ assertTrue(!!nameField.disabled);
+ });
+
// Tests profile name updates pushed from the browser.
test('ManageProfileNameUpdated', function() {
var nameField = manageProfile.$.name;
« no previous file with comments | « chrome/browser/resources/settings/people_page/people_page.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698