Plugins page in BWSI session has unnecessary and misleading UI components

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 24 matching lines @@
       // Set up accounts page.
       var userList = $('userList');
       userList.addEventListener('remove', this.handleRemoveUser_);
 
       var userNameEdit = $('userNameEdit');
       options.accounts.UserNameEdit.decorate(userNameEdit);
       userNameEdit.addEventListener('add', this.handleAddUser_);
 
       // If the current user is not the owner, show some warning,
       // and do not show the user list.
-      this.showWhitelist_ = AccountsOptions.currentUserIsOwner();
+      this.showWhitelist_ = UIAccountTweaks.currentUserIsOwner();
       if (this.showWhitelist_) {
         options.accounts.UserList.decorate(userList);
       } else {
         if (!AccountsOptions.whitelistIsManaged()) {
           $('ownerOnlyWarning').hidden = false;
         } else {
           this.managed = true;
         }
       }
 
@@ skipping 67 matching lines @@
     /**
      * Handler for "remove" event fired from userList.
      * @private
      * @param {Event} e Remove event fired from userList.
      */
     handleRemoveUser_: function(e) {
       chrome.send('unwhitelistUser', [e.user.username]);
     }
   };
 
-  /**
-   * 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('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 or hide some elements in Guest mode in ChromeOS.
-   * All elements within given document with guest-visibility
-   * attribute are either hidden (for guest-visibility="hidden")
-   * or disabled (for guest-visibility="disabled").
-   *
-   * @param {Document} document Document that should processed.
-   */
-  AccountsOptions.applyGuestModeVisibility = function(document) {
-    if (!cr.isChromeOS || !AccountsOptions.loggedInAsGuest())
-      return;
-    var elements = document.querySelectorAll('[guest-visibility]');
-    for (var i = 0; i < elements.length; i++) {
-      var element = elements[i];
-      var visibility = element.getAttribute('guest-visibility');
-      if (visibility == 'hidden') {
-        element.hidden = true;
-      } else if (visibility == 'disabled') {
-        AccountsOptions.disableElementsForGuest(element);
-      }
-    }
-  }
-
-  /**
-   * Disables and marks page elements for Guest mode.
-   * Adds guest-disabled css class to all elements within given subtree,
-   * 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.
-   * Adds guest-disabled css class, adds disabled attribute for appropriate
-   * 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
   };
 
 });