PART2: Make SignedSettings use proper Value types instead of string all around the place.

chrome/browser/chromeos/login/existing_user_controller.cc

 // 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.
 
 #include "chrome/browser/chromeos/login/existing_user_controller.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 84 matching lines @@
 void ExistingUserController::Init(const UserList& users) {
   UserList filtered_users;
   bool show_users_on_signin;
 
   // TODO(pastarmovj): Make this class an observer of the CrosSettings to be
   // able to update the UI whenever policy is loaded.
   cros_settings_->GetBoolean(kAccountsPrefShowUserNamesOnSignIn,
                              &show_users_on_signin);
   if (show_users_on_signin) {
     bool allow_new_user = false;
-    const base::ListValue *user_list;
     cros_settings_->GetBoolean(kAccountsPrefAllowNewUser, &allow_new_user);
-    cros_settings_->GetList(kAccountsPrefUsers, &user_list);
     for (UserList::const_iterator it = users.begin(); it != users.end(); ++it) {
-      base::StringValue email((*it)->email());
       // TODO(xiyuan): Clean user profile whose email is not in whitelist.
       if (allow_new_user ||
-          user_list->Find(email) != user_list->end()) {
+          cros_settings_->FindEmailInList(kAccountsPrefUsers,
+                                          (*it)->email())) {
         filtered_users.push_back(*it);
       }
     }
   }
 
   // If no user pods are visible, fallback to single new user pod which will
   // have guest session link.
   bool show_guest;
   cros_settings_->GetBoolean(kAccountsPrefAllowGuest, &show_guest);
   show_guest &= !filtered_users.empty();
@@ skipping 487 matching lines @@
   std::string owner;
   cros_settings_->GetString(kDeviceOwner, &owner);
   cryptohomed->AsyncSetOwnerUser(owner, NULL);
 
   // Do not invoke AsyncDoAutomaticFreeDiskSpaceControl(NULL) here
   // so it does not delay the following mount. Cleanup will be
   // started in Cryptohomed by timer.
 }
 
 }  // namespace chromeos