Browser-based guest policy implementation

chrome/browser/resources/user_manager/control_bar.js

Index: chrome/browser/resources/user_manager/control_bar.js
diff --git a/chrome/browser/resources/user_manager/control_bar.js b/chrome/browser/resources/user_manager/control_bar.js
index 2fe4ac45c2fb38413c9b48de96f8d0e2b967a5a6..257481438e9278fafca1362ddb605926efa27990 100644
--- a/chrome/browser/resources/user_manager/control_bar.js
+++ b/chrome/browser/resources/user_manager/control_bar.js
@@ -26,6 +26,15 @@ cr.define('login', function() {
     // 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',
@@ -90,15 +99,6 @@ cr.define('login', function() {
     },
 
     /**
-     * Guest button click handler.
-     * @private
-     */
-    handleGuestClick_: function(e) {
-      chrome.send('launchGuest');
-      e.stopPropagation();
-    },
-
-    /**
      * If true then "Browse as Guest" button is shown.
      * @type {boolean}
      */
@@ -135,6 +135,14 @@ cr.define('login', function() {
       $('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;
+      }
     },
 
     /**