diff options
author | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 19:40:27 +0000 |
---|---|---|
committer | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-01 19:40:27 +0000 |
commit | 3bba9de6d9097f02102ee3f8e7f6ab56f8ab31a7 (patch) | |
tree | adcf38f9b0ac24e8305a750265a653d65d3b67c3 | |
parent | d2b32d755d34e63659ce67a387d2c3ac740c4226 (diff) | |
download | chromium_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.js | 8 |
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'); } |