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

chrome/browser/ui/webui/options/chromeos/accounts_options_handler.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/ui/webui/options/chromeos/accounts_options_handler.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/json/json_reader.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 83 matching lines @@
     return;
   }
   // TODO(pastarmovj): Those will change to CrosSettings ops in phase 2.
   UserCrosSettingsProvider::UnwhitelistUser(Authenticator::Canonicalize(email));
   UserManager::Get()->RemoveUser(email, NULL);
 }
 
 void AccountsOptionsHandler::WhitelistExistingUsers(
     const base::ListValue* args) {
   base::ListValue whitelist_users;
-  const base::ListValue *user_list;
-  CrosSettings::Get()->GetList(kAccountsPrefUsers, &user_list);
   UserVector users = UserManager::Get()->GetUsers();
   for (UserVector::const_iterator it = users.begin();
        it < users.end(); ++it) {
     const std::string& email = it->email();
-    base::StringValue email_value(email);
-    if (user_list->Find(email_value) == user_list->end()) {
+    if (CrosSettings::Get()->FindEmailInList(kAccountsPrefUsers, email)) {
       base::DictionaryValue* user_dict = new DictionaryValue;
       user_dict->SetString("name", it->GetDisplayName());
       user_dict->SetString("email", email);
       user_dict->SetBoolean("owner", false);
 
       whitelist_users.Append(user_dict);
     }
   }
 
   web_ui_->CallJavascriptFunction("AccountsOptions.addUsers", whitelist_users);
 }
 
 }  // namespace chromeos