Hide/Disable several (meaningless) options in Settings uber-page for Guest.

chrome/browser/resources/options2/chromeos/accounts_options.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options', function() {
   var OptionsPage = options.OptionsPage;
 
   /////////////////////////////////////////////////////////////////////////////
   // AccountsOptions class:
 
@@ skipping 123 matching lines @@
    * Returns whether the current user is owner or not.
    */
   AccountsOptions.currentUserIsOwner = function() {
     return localStrings.getString('current_user_is_owner') == 'true';
   };
 
   /**
    * Returns whether we're currently in guest mode.
    */
   AccountsOptions.loggedInAsGuest = function() {
-    return localStrings.getString('logged_in_as_guest') == 'true';
+    return localStrings.getString('loggedInAsGuest') == 'true';
   };
 
   /**
    * Returns whether the whitelist is managed by policy or not.
    */
   AccountsOptions.whitelistIsManaged = function() {
     return localStrings.getString('whitelist_is_managed') == 'true';
   };
 
   /**
    * Update account picture.
    * @param {string} username User for which to update the image.
    */
   AccountsOptions.updateAccountPicture = function(username) {
     if (this.showWhitelist_)
       $('userList').updateAccountPicture(username);
   };
 
+  /**
+   * Disable and mark page elements for Guest mode.

[James Hawkins, 2012/03/05 17:07:12]

nit: Method documentation should be declarative (as opposed to imperative, used
here).

"Disables and marks page elements for Guest mode."

[Denis Kuznetsov (DE-MUC), 2012/03/12 10:32:15]

Done.

+   * It adds guest-disabled css class to all elements within given subtree,

[James Hawkins, 2012/03/05 17:07:12]

nit: Don't use pronouns (it) in comments; they're ambiguous.  It's also shorter
w/out:

"Adds guest-disabled..."

[Denis Kuznetsov (DE-MUC), 2012/03/12 10:32:15]

Done.

+   * disables interactive elements (input/select/button), and removes href
+   * attribute from <a> elements.
+   *
+   * @param {Element} element Root element of DOM subtree that should be
+   *     disabled.
+   */
+  AccountsOptions.disableElementsForGuest = function(element) {
+    AccountsOptions.disableElementForGuest_(element);
+
+    // Walk the tree, searching each ELEMENT node.
+    var walker = document.createTreeWalker(element,
+                                           NodeFilter.SHOW_ELEMENT,
+                                           null,
+                                           false);
+
+    var node = walker.nextNode();
+    while (node) {
+      AccountsOptions.disableElementForGuest_(node);
+      node = walker.nextNode();
+    }
+  };
+
+  /**
+   * Disables single element for Guest mode.
+   * It adds guest-disabled css class, adds disabled attribute for appropriate

[James Hawkins, 2012/03/05 17:07:12]

nit: s/It adds/Adds/

[Denis Kuznetsov (DE-MUC), 2012/03/12 10:32:15]

Done.

+   * elements (input/select/button), and removes href attribute from
+   * <a> element.
+   *
+   * @private
+   * @param {Element} element Element that should be disabled.
+   */
+  AccountsOptions.disableElementForGuest_ = function(element) {
+    element.classList.add('guest-disabled');
+    if (element.nodeName == 'INPUT' ||
+        element.nodeName == 'SELECT' ||
+        element.nodeName == 'BUTTON')
+      element.disabled = true;
+    if (element.nodeName == 'A')
+      element.removeAttribute('href');
+  };
+
   // Export
   return {
     AccountsOptions: AccountsOptions
   };
 
 });