[cros] Display emails of users are stored in a separate dictionary in Local State.

chrome/browser/resources/options/chromeos/accounts_user_list.js

 // Copyright (c) 2011 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.accounts', function() {
   const List = cr.ui.List;
   const ListItem = cr.ui.ListItem;
   const ArrayDataModel = cr.ui.ArrayDataModel;
 
   /**
@@ skipping 25 matching lines @@
           function(event) {
             self.load_(event.value);
           });
     },
 
     createItem: function(user) {
       return new UserListItem(user);
     },
 
     /**
-     * Finds the index of user by given email.
+     * Finds the index of user by given username (canonicalized email).
      * @private
-     * @param {string} email The email address to look for.
+     * @param {string} username The username to look for.
      * @return {number} The index of the found user or -1 if not found.
      */
-    findUserByEmail_: function(email) {
+    indexOf_: function(username) {
       var dataModel = this.dataModel;
       if (!dataModel)
         return -1;
 
       var length = dataModel.length;
       for (var i = 0; i < length; ++i) {
         var user = dataModel.item(i);
-        if (user.email == email) {
+        if (user.username == username) {
           return i;
         }
       }
 
       return -1;
     },
 
     /**
-     * Adds given user to model and update backend.
-     * @param {Object} user A user to be added to user list.
+     * Adds given user to the model (this comes from the backend so there is no
+     * need to update it).
+     * @param {Object} user User info object to be added to user list.
      */
     addUser: function(user) {
-      var index = this.findUserByEmail_(user.email);
-      if (index == -1) {
-        this.dataModel.push(user);
-        chrome.send('whitelistUser', [user.email]);
-      }
+      this.dataModel.push(user);
     },
 
     /**
-     * Removes given user from model and update backend.
+     * Removes given user from the tmodel and updates backend.

[James Hawkins, 2011/12/02 18:12:01]

tmodel: is this a spelling mistake?

[Ivan Korotkov, 2011/12/02 18:30:30]

Yep, fixed.

+     * @param {Object} user User info object to be removed from user list.
      */
     removeUser: function(user) {
       var dataModel = this.dataModel;
-
       var index = dataModel.indexOf(user);
       if (index >= 0) {
         dataModel.splice(index, 1);
-        chrome.send('unwhitelistUser', [user.email]);
+        chrome.send('unwhitelistUser', [user.username]);
       }
     },
 
     /**
      * Update given user's account picture.
-     * @param {string} email Email of the user to update.
+     * @param {string} username User for which to update the image.
      */
-    updateAccountPicture: function(email) {
-      var index = this.findUserByEmail_(email);
+    updateAccountPicture: function(username) {
+      var index = this.indexOf_(username);
       if (index >= 0) {
         var item = this.getListItemByIndex(index);
         if (item)
           item.updatePicture();
       }
     },
 
     /**
      * Handles the clicks on the list and triggers user removal if the click
      * is on the remove user button.
@@ skipping 86 matching lines @@
         removeButton.className =
             'raw-button remove-user-button custom-appearance';
         this.appendChild(removeButton);
       }
     },
 
     /**
      * Reloads user picture.
      */
     updatePicture: function() {
-      this.icon_.src = 'chrome://userimage/' + this.user.email +
+      this.icon_.src = 'chrome://userimage/' + this.user.username +
                        '?id=' + (new Date()).getTime();
     }
   };
 
   return {
     UserList: UserList
   };
 });