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

Unified Diff: components/signin/core/browser/account_tracker_service_unittest.cc

Issue 1091363002: Change ProfileDownloader to use AccountTrackerService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits. Created 5 years, 7 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
« no previous file with comments | « components/signin/core/browser/account_tracker_service.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/signin/core/browser/account_tracker_service_unittest.cc
diff --git a/components/signin/core/browser/account_tracker_service_unittest.cc b/components/signin/core/browser/account_tracker_service_unittest.cc
index fe4a929915c54d1cd705397c2235a113809c90b4..90ffab8aedfe8c5fec076575b32f1ab71f72dc07 100644
--- a/components/signin/core/browser/account_tracker_service_unittest.cc
+++ b/components/signin/core/browser/account_tracker_service_unittest.cc
@@ -29,7 +29,8 @@ const std::string kTokenInfoResponseFormat =
\"hd\": \"\", \
\"name\": \"%s\", \
\"given_name\": \"%s\", \
- \"locale\": \"%s\" \
+ \"locale\": \"%s\", \
+ \"picture\": \"%s\" \
}";
const std::string kTokenInfoIncompleteResponseFormat =
@@ -68,6 +69,10 @@ std::string AccountIdToLocale(const std::string account_id) {
return "locale-" + account_id;
}
+std::string AccountIdToPictureURL(const std::string account_id) {
+ return "picture_url-" + account_id;
+}
+
void CheckAccountDetails(const std::string account_id,
const AccountTrackerService::AccountInfo& info) {
EXPECT_EQ(account_id, info.account_id);
@@ -291,7 +296,8 @@ class AccountTrackerServiceTest : public testing::Test {
AccountIdToEmail(account_id).c_str(),
AccountIdToFullName(account_id).c_str(),
AccountIdToGivenName(account_id).c_str(),
- AccountIdToLocale(account_id).c_str());
+ AccountIdToLocale(account_id).c_str(),
+ AccountIdToPictureURL(account_id).c_str());
}
std::string GenerateIncompleteTokenInfoResponse(
« no previous file with comments | « components/signin/core/browser/account_tracker_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698