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

Unified Diff: chrome/browser/resources/options/chromeos_accounts_options.js

Issue 5965003: Show warning text in the user settings page for non-owner users. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments Created 10 years 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
Index: chrome/browser/resources/options/chromeos_accounts_options.js
diff --git a/chrome/browser/resources/options/chromeos_accounts_options.js b/chrome/browser/resources/options/chromeos_accounts_options.js
index 349f15b45cb97f43392e258e4a448bddaf98bd32..0695d7b281b7039fbb8d0bb81d49fcbbec07d217 100644
--- a/chrome/browser/resources/options/chromeos_accounts_options.js
+++ b/chrome/browser/resources/options/chromeos_accounts_options.js
@@ -41,6 +41,12 @@ cr.define('options', function() {
userList.disabled =
userNameEdit.disabled = !AccountsOptions.currentUserIsOwner();
+ // If the current user is not the owner, show some warning.
+ if (!AccountsOptions.currentUserIsOwner()) {
+ $('ownerOnlyWarning')..classList.remove('hidden');
arv (Not doing code reviews) 2010/12/20 19:03:36 syntax error
satorux1 2010/12/21 01:57:17 Done.
+ $('ownerUserId').textContent =
+ localStrings.getString('owner_user_id');
arv (Not doing code reviews) 2010/12/20 19:03:36 Can you move this to markup?
satorux1 2010/12/21 01:57:17 Done.
+ }
this.addEventListener('visibleChange', this.handleVisibleChange_);

Powered by Google App Engine
This is Rietveld 408576698