Chromium Code Reviews| Index: chrome/browser/resources/options/manage_profile_overlay.js |
| diff --git a/chrome/browser/resources/options/manage_profile_overlay.js b/chrome/browser/resources/options/manage_profile_overlay.js |
| index df9862b545f1c027297dbf3f5fed425fdef660cf..a96a2713bc32b8a20dec2fac09c5c8ca988f5ba8 100644 |
| --- a/chrome/browser/resources/options/manage_profile_overlay.js |
| +++ b/chrome/browser/resources/options/manage_profile_overlay.js |
| @@ -379,22 +379,31 @@ cr.define('options', function() { |
| var newName = $('create-profile-name').value; |
| var i; |
| for (i = 0; i < supervisedUsers.length; ++i) { |
| - if (supervisedUsers[i].name == newName && |
| - !supervisedUsers[i].onCurrentDevice) { |
| - var errorHtml = loadTimeData.getStringF( |
| - 'manageProfilesExistingSupervisedUser', |
| - HTMLEscape(elide(newName, /* maxLength */ 50))); |
| - this.showErrorBubble_(errorHtml, 'create', true); |
| - |
| + if (supervisedUsers[i].name == newName) { |
| // Check if another supervised user also exists with that name. |
| var nameIsUnique = true; |
| + // Handling the case when multiple SU user with the same |
|
Pam (message me for reviews)
2015/12/15 22:20:13
nit:
user with the same name are created
->
users
atanasova
2015/12/16 09:21:13
Done.
|
| + // name are created, but not all of them are on the device. |
| + // If at least one is not imported, we want to offer that |
| + // option to the user. |
| + var allOnCurrentDevice = supervisedUsers[i].onCurrentDevice; |
| var j; |
| for (j = i + 1; j < supervisedUsers.length; ++j) { |
| if (supervisedUsers[j].name == newName) { |
| nameIsUnique = false; |
| - break; |
| + allOnCurrentDevice = allOnCurrentDevice && |
| + supervisedUsers[j].onCurrentDevice; |
| } |
| } |
| + |
| + var errorHtml = allOnCurrentDevice ? |
| + loadTimeData.getStringF( |
| + 'managedProfilesExistingLocalSupervisedUser') : |
| + loadTimeData.getStringF( |
| + 'manageProfilesExistingSupervisedUser', |
| + HTMLEscape(elide(newName, /* maxLength */ 50))); |
| + this.showErrorBubble_(errorHtml, 'create', true); |
| + |
| $('supervised-user-import-existing').onclick = |
| this.getImportHandler_(supervisedUsers[i], nameIsUnique); |
| $('create-profile-ok').disabled = true; |