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

Unified Diff: chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc

Issue 2471993002: Remove calls to User::email() from chrome/browser/ui/webui/* (Closed)
Patch Set: Created 4 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/chromeos/login/signin_userlist_unittest.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc b/chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc
index 36ddad29b9318929aa35f04eb4eb1a13c4ccdddf..c19b3ebe63cd8f6327fad8aa4c9610c8873aa901 100644
--- a/chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc
+++ b/chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc
@@ -94,7 +94,7 @@ TEST_F(SigninPrepareUserListTest, AlwaysKeepOwnerInList) {
true /* is signin to add */);
EXPECT_EQ(kMaxUsers, users_to_send.size());
- EXPECT_EQ(kOwner, users_to_send.back()->email());
+ EXPECT_EQ(kOwner, users_to_send.back()->GetAccountId().GetUserEmail());
fake_user_manager_->RemoveUserFromList(
AccountId::FromUserEmail("a16@gmail.com"));
@@ -105,8 +105,10 @@ TEST_F(SigninPrepareUserListTest, AlwaysKeepOwnerInList) {
true /* is signin to add */);
EXPECT_EQ(kMaxUsers, users_to_send.size());
- EXPECT_EQ("a18@gmail.com", users_to_send.back()->email());
- EXPECT_EQ(kOwner, users_to_send[kMaxUsers-2]->email());
+ EXPECT_EQ("a18@gmail.com",
+ users_to_send.back()->GetAccountId().GetUserEmail());
+ EXPECT_EQ(kOwner,
+ users_to_send[kMaxUsers - 2]->GetAccountId().GetUserEmail());
}
TEST_F(SigninPrepareUserListTest, PublicAccounts) {
@@ -116,14 +118,16 @@ TEST_F(SigninPrepareUserListTest, PublicAccounts) {
true /* is signin to add */);
EXPECT_EQ(kMaxUsers, users_to_send.size());
- EXPECT_EQ("a0@gmail.com", users_to_send.front()->email());
+ EXPECT_EQ("a0@gmail.com",
+ users_to_send.front()->GetAccountId().GetUserEmail());
users_to_send = UserSelectionScreen::PrepareUserListForSending(
fake_user_manager_->GetUsers(), AccountId::FromUserEmail(kOwner),
false /* is signin to add */);
EXPECT_EQ(kMaxUsers, users_to_send.size());
- EXPECT_EQ("public0@gmail.com", users_to_send.front()->email());
+ EXPECT_EQ("public0@gmail.com",
+ users_to_send.front()->GetAccountId().GetUserEmail());
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698