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

Unified Diff: chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js

Issue 256223003: Supervised users import: make sure users are up-to-date. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes Created 6 years, 8 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/chromeos/login/managed/locally_managed_user_creation_screen.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js
diff --git a/chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js b/chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js
index 8a8d224e3cfba7bbfd6ec3ae06ff7d95786fd4ef..75040fd76cb3ccfbc52cc9863146a9f3e4c88728 100644
--- a/chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js
+++ b/chrome/browser/resources/chromeos/login/screen_locally_managed_user_creation.js
@@ -370,6 +370,14 @@ login.createScreen('LocallyManagedUserCreationScreen',
return this.children;
},
+ /**
+ * Returns selected pod.
+ * @type {Node}
+ */
+ get selectedPod() {
+ return this.selectedPod_;
+ },
+
addPod: function(user) {
var importPod = new ImportPod({user: user});
this.appendChild(importPod);
@@ -835,6 +843,7 @@ login.createScreen('LocallyManagedUserCreationScreen',
if (this.disabled)
return;
this.disabled = true;
+
this.context_.managedName = userName;
chrome.send('specifyLocallyManagedUserCreationFlowUserData',
[userName, firstPassword]);
@@ -1301,6 +1310,7 @@ login.createScreen('LocallyManagedUserCreationScreen',
control.disabled = value;
}
$('login-header-bar').disabled = value;
+ $('cancel-add-user-button').disabled = false;
},
/**
@@ -1542,8 +1552,12 @@ login.createScreen('LocallyManagedUserCreationScreen',
},
setExistingManagedUsers: function(users) {
- var userList = users;
+ var selectedUser = null;
+ // Store selected user
+ if (this.importList_.selectedPod)
+ selectedUser = this.importList_.selectedPod.user.id;
+ var userList = users;
userList.sort(function(a, b) {
// Put existing users last.
if (a.exists != b.exists)
@@ -1553,14 +1567,21 @@ login.createScreen('LocallyManagedUserCreationScreen',
});
this.importList_.clearPods();
- for (var i = 0; i < userList.length; ++i)
+ var selectedIndex = -1;
+ for (var i = 0; i < userList.length; ++i) {
this.importList_.addPod(userList[i]);
+ if (selectedUser == userList[i].id)
+ selectedIndex = i;
+ }
if (userList.length == 1)
- this.importList_.selectPod(this.managerList_.pods[0]);
+ this.importList_.selectPod(this.importList_.pods[0]);
+
+ if (selectedIndex >= 0)
+ this.importList_.selectPod(this.importList_.pods[selectedIndex]);
- if (userList.length > 0 && this.currentPage_ == 'username')
- this.getScreenElement('import-link').hidden = false;
+ if (this.currentPage_ == 'username')
+ this.getScreenElement('import-link').hidden = (userList.length == 0);
},
};
});
« no previous file with comments | « chrome/browser/chromeos/login/managed/locally_managed_user_creation_screen.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698