Browser-based guest policy implementation

chrome/browser/resources/user_manager/control_bar.js

 // Copyright 2013 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.
 
 /**
  * @fileoverview Desktop User Chooser UI control bar implementation.
  */
 
 cr.define('login', function() {
   /**
    * Creates a header bar element.
    * @constructor
    * @extends {HTMLDivElement}
    */
   var HeaderBar = cr.ui.define('div');
 
   HeaderBar.prototype = {
     __proto__: HTMLDivElement.prototype,
 
     // Whether guest button should be shown when header bar is in normal mode.
     showGuest_: true,
 
     // Current UI state of the sign-in screen.
     signinUIState_: SIGNIN_UI_STATE.ACCOUNT_PICKER,
 
     // Whether to show kiosk apps menu.
     hasApps_: false,
 
+    /**
+     * Guest button click handler.
+     * @private
+     */
+    handleGuestClick_: function(e) {
+      chrome.send('launchGuest');
+      e.stopPropagation();
+    },
+
     /** @override */
     decorate: function() {
       $('add-user-button').addEventListener('click',
           this.handleAddUserClick_);
       $('cancel-add-user-button').addEventListener('click',
           this.handleCancelAddUserClick_);
       $('guest-user-header-bar-item').addEventListener('click',
           this.handleGuestClick_);
       $('guest-user-button').addEventListener('click',
           this.handleGuestClick_);
@@ skipping 44 matching lines @@
           Oobe.getInstance().currentScreen.cancel) {
         Oobe.getInstance().currentScreen.cancel();
         return;
       }
 
       Oobe.showScreen({id: SCREEN_ACCOUNT_PICKER});
       Oobe.resetSigninUI(true);
     },
 
     /**
-     * Guest button click handler.
-     * @private
-     */
-    handleGuestClick_: function(e) {
-      chrome.send('launchGuest');
-      e.stopPropagation();
-    },
-
-    /**
      * If true then "Browse as Guest" button is shown.
      * @type {boolean}
      */
     set showGuestButton(value) {
       this.showGuest_ = value;
       this.updateUI_();
     },
 
     /**
      * Update current header bar UI.
@@ skipping 16 matching lines @@
 
     /**
      * Updates visibility state of action buttons.
      * @private
      */
     updateUI_: function() {
       $('add-user-button').hidden = false;
       $('cancel-add-user-button').hidden = !this.allowCancel_;
       $('guest-user-header-bar-item').hidden = false;
       $('add-user-header-bar-item').hidden = false;
+      //$('guest-user-button').hidden = !this.showGuest_;

[Joao da Silva, 2014/07/14 17:11:59]

Remove this?

[Mike Lerman, 2014/07/15 14:28:12]

Still working on UI... this method has some more changes to come.

+      if (!this.showGuest_) {
+        $('guest-user-header-bar-item').removeEventListener('click',
+              this.handleGuestClick_);
+        $('guest-user-button').removeEventListener('click',
+              this.handleGuestClick_);
+        $('guest-user-button').disabled = true;
+      }
     },
 
     /**
      * Animates Header bar to hide from the screen.
      * @param {function()} callback will be called once animation is finished.
      */
     animateOut: function(callback) {
       var launcher = this;
       launcher.addEventListener(
           'webkitTransitionEnd', function f(e) {
@@ skipping 26 matching lines @@
    * Convenience wrapper of animateIn.
    */
   HeaderBar.animateIn = function() {
     $('login-header-bar').animateIn();
   }
 
   return {
     HeaderBar: HeaderBar
   };
 });