summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-02-01 19:40:27 +0000
committerxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-02-01 19:40:27 +0000
commit3bba9de6d9097f02102ee3f8e7f6ab56f8ab31a7 (patch)
treeadcf38f9b0ac24e8305a750265a653d65d3b67c3
parentd2b32d755d34e63659ce67a387d2c3ac740c4226 (diff)
downloadchromium_src-3bba9de6d9097f02102ee3f8e7f6ab56f8ab31a7.zip
chromium_src-3bba9de6d9097f02102ee3f8e7f6ab56f8ab31a7.tar.gz
chromium_src-3bba9de6d9097f02102ee3f8e7f6ab56f8ab31a7.tar.bz2
Show user list only to owner (settings->users).
User list is still visible, but grayed out and empty. Land http://codereview.chromium.org/6250079/ for dgozman@chromium.org. BUG=chromium-os:9078 TEST=Observe empty user list in settings->users for non-owner. Review URL: http://codereview.chromium.org/6368029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73326 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/resources/options/chromeos_accounts_options.js8
1 files changed, 5 insertions, 3 deletions
diff --git a/chrome/browser/resources/options/chromeos_accounts_options.js b/chrome/browser/resources/options/chromeos_accounts_options.js
index 1a978c6..d0d99a2 100644
--- a/chrome/browser/resources/options/chromeos_accounts_options.js
+++ b/chrome/browser/resources/options/chromeos_accounts_options.js
@@ -33,7 +33,6 @@ cr.define('options', function() {
// Set up accounts page.
var userList = $('userList');
- options.accounts.UserList.decorate(userList);
var userNameEdit = $('userNameEdit');
options.accounts.UserNameEdit.decorate(userNameEdit);
@@ -41,8 +40,11 @@ cr.define('options', function() {
userList.disabled =
userNameEdit.disabled = !AccountsOptions.currentUserIsOwner();
- // If the current user is not the owner, show some warning.
- if (!AccountsOptions.currentUserIsOwner()) {
+ // If the current user is not the owner, show some warning,
+ // and do not show the user list.
+ if (AccountsOptions.currentUserIsOwner()) {
+ options.accounts.UserList.decorate(userList);
+ } else {
$('ownerOnlyWarning').classList.remove('hidden');
}