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

Unified Diff: ash/system/user/tray_user_unittest.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: Update after review. Created 5 years, 2 months 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: ash/system/user/tray_user_unittest.cc
diff --git a/ash/system/user/tray_user_unittest.cc b/ash/system/user/tray_user_unittest.cc
index 5d9d3d3ab9bc6ee74843aabc55f84bedf4aa4448..ee1a92aabcb0ab9e98dc49ada21e8d5365bf6c61 100644
--- a/ash/system/user/tray_user_unittest.cc
+++ b/ash/system/user/tray_user_unittest.cc
@@ -17,6 +17,7 @@
#include "ash/test/test_session_state_delegate.h"
#include "ash/test/test_shell_delegate.h"
#include "base/strings/utf_string_conversions.h"
+#include "components/signin/core/account_id/account_id.h"
#include "components/user_manager/user_info.h"
#include "ui/accessibility/ax_view_state.h"
#include "ui/events/test/event_generator.h"
@@ -272,10 +273,11 @@ TEST_F(TrayUserTest, MutiUserModeButtonClicks) {
ClickUserItem(&generator, 1);
const user_manager::UserInfo* active_user = delegate()->GetActiveUserInfo();
const user_manager::UserInfo* second_user = delegate()->GetUserInfo(1);
- EXPECT_EQ(active_user->GetUserID(), second_user->GetUserID());
+ EXPECT_EQ(active_user->GetAccountId(), second_user->GetAccountId());
// Since the name is capitalized, the email should be different then the
// user_id.
- EXPECT_NE(active_user->GetUserID(), second_user->GetEmail());
+ EXPECT_NE(active_user->GetAccountId().GetUserEmail(),
+ second_user->GetEmail());
tray()->CloseSystemBubble();
}

Powered by Google App Engine
This is Rietveld 408576698