Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1822)

Unified Diff: chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Fix Win GN build. Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
index 33c6c55c3ee838816036ce5a6b2d094e5c539bae..3dcb20f966cbfa4e9e67dac45026a1d30a8b485e 100644
--- a/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
+++ b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
@@ -37,6 +37,9 @@ class AccountsOptionsTest : public LoginManagerTest {
: LoginManagerTest(false),
device_settings_provider_(NULL) {
stub_settings_provider_.Set(kDeviceOwner, base::StringValue(kTestUsers[0]));
+ for (size_t i = 0; i < arraysize(kTestUsers); ++i) {
+ test_users_.push_back(AccountId::FromUserEmail(kTestUsers[i]));
+ }
}
~AccountsOptionsTest() override {}
@@ -117,28 +120,29 @@ class AccountsOptionsTest : public LoginManagerTest {
StubCrosSettingsProvider stub_settings_provider_;
CrosSettingsProvider* device_settings_provider_;
+ std::vector<AccountId> test_users_;
private:
DISALLOW_COPY_AND_ASSIGN(AccountsOptionsTest);
};
IN_PROC_BROWSER_TEST_F(AccountsOptionsTest, PRE_MultiProfilesAccountsOptions) {
- RegisterUser(kTestUsers[0]);
- RegisterUser(kTestUsers[1]);
+ RegisterUser(test_users_[0].GetUserEmail());
+ RegisterUser(test_users_[1].GetUserEmail());
StartupUtils::MarkOobeCompleted();
}
IN_PROC_BROWSER_TEST_F(AccountsOptionsTest, MultiProfilesAccountsOptions) {
- LoginUser(kTestUsers[0]);
+ LoginUser(test_users_[0].GetUserEmail());
UserAddingScreen::Get()->Start();
content::RunAllPendingInMessageLoop();
- AddUser(kTestUsers[1]);
+ AddUser(test_users_[1].GetUserEmail());
user_manager::UserManager* manager = user_manager::UserManager::Get();
ASSERT_EQ(2u, manager->GetLoggedInUsers().size());
- CheckAccountsUI(manager->FindUser(kTestUsers[0]), true /* is_owner */);
- CheckAccountsUI(manager->FindUser(kTestUsers[1]), false /* is_owner */);
+ CheckAccountsUI(manager->FindUser(test_users_[0]), true /* is_owner */);
+ CheckAccountsUI(manager->FindUser(test_users_[1]), false /* is_owner */);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698