First try at a user management screen for the desktop

chrome/browser/resources/chromeos/login/user_pod_row.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.
 
 /**
  * @fileoverview User pod row implementation.
  */
 
 cr.define('login', function() {
   /**
@@ skipping 281 matching lines @@
      * @type {!HTMLInputElement}
      */
     get actionBoxMenuRemoveElement() {
       return this.querySelector('.action-box-menu-remove');
     },
 
     /**
      * Updates the user pod element.
      */
     update: function() {
-      this.imageElement.src = 'chrome://userimage/' + this.user.username +
-          '?id=' + UserPod.userImageSalt_[this.user.username];
+      if (this.user.isDesktopScreen) {
+        this.imageElement.src = this.user.userImage;
+      } else {
+        this.imageElement.src = 'chrome://userimage/' + this.user.username +
+            '?id=' + UserPod.userImageSalt_[this.user.username];
+      }

[Roger Tawa OOO till Jul 10th, 2013/06/05 18:53:56]

Would it be possible to derive something like DesktopUserPod from UserPod so
there is no boolean?

[noms, 2013/06/06 19:01:36]

Done.

 
       this.nameElement.textContent = this.user_.displayName;
       this.actionBoxAreaElement.hidden = this.user_.publicAccount;
       this.actionBoxMenuRemoveElement.hidden = !this.user_.canRemove;
       this.signedInIndicatorElement.hidden = !this.user_.signedIn;
 
       var needSignin = this.needGaiaSignin;
       this.passwordElement.hidden = needSignin;
       this.actionBoxAreaElement.setAttribute(
           'aria-label', loadTimeData.getStringF(
@@ skipping 113 matching lines @@
       this.tabIndex = -1;
       this.mainInput.tabIndex = UserPodTabOrder.POD_INPUT;
       this.mainInput.focus();
     },
 
     /**
      * Activates the pod.
      * @return {boolean} True if activated successfully.
      */
     activate: function() {
+      if (this.user.isDesktopScreen) {
+        Oobe.launchUser(this.user.emailAddress,
+                        this.user.displayName);
+        return;
+      }
+
       if (!this.signinButtonElement.hidden) {
         // Switch to Gaia signin.
         this.showSigninUI();
       } else if (!this.passwordElement.value) {
         return false;
       } else {
         Oobe.disableSigninUI();
         chrome.send('authenticateUser',
                     [this.user.username, this.passwordElement.value]);
       }
@@ skipping 112 matching lines @@
       this.actionBoxMenuRemoveElement.tabIndex = -1;
     },
 
     /**
      * Handles mousedown event on a user pod.
      * @param {Event} e Mousedown event.
      */
     handleMouseDown_: function(e) {
       if (this.parentNode.disabled)
         return;
+
+      // We don't want to sign in until the user presses
+      // the button. Just activate the pod.
+      if (this.user.isDesktopScreen) {
+        Oobe.clearErrors();
+        this.disabled = true;
+        this.parentNode.lastFocusedPod_ =
+            this.parentNode.getPodWithUsername_(this.user.emailAddress);
+        return;
+      }
+

[Roger Tawa OOO till Jul 10th, 2013/06/05 18:53:56]

Should probably move this file to a directory that is not chromeos specific.

[noms, 2013/06/06 19:01:36]

This file is actually part of pretty large codebase (it's used by the login
page, which is included in the oobe flow for CrOS), so we can't really move just
it out, and I don't think we should move all of the flow out of the directory...

On 2013/06/05 18:53:56, Roger Tawa wrote:

       if (!this.signinButtonElement.hidden && !this.isActionBoxMenuActive) {
         this.showSigninUI();
         // Prevent default so that we don't trigger 'focus' event.
         e.preventDefault();
       }
     }
   };
 
   /**
    * Creates a public account user pod.
@@ skipping 824 matching lines @@
         this.classList.remove('images-loading');
         chrome.send('userImagesLoaded');
       }
     }
   };
 
   return {
     PodRow: PodRow
   };
 });