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

Unified Diff: chrome/browser/chromeos/login/auth/online_attempt_unittest.cc

Issue 290483003: Tame the proliferation of UserContext constructors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased. Created 6 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
Index: chrome/browser/chromeos/login/auth/online_attempt_unittest.cc
diff --git a/chrome/browser/chromeos/login/auth/online_attempt_unittest.cc b/chrome/browser/chromeos/login/auth/online_attempt_unittest.cc
index 5176a3d307630881fd1719f31a6ec22aa3a123d2..85e93f6c6af5ca1b04f3bee31cdc14aacc012431 100644
--- a/chrome/browser/chromeos/login/auth/online_attempt_unittest.cc
+++ b/chrome/browser/chromeos/login/auth/online_attempt_unittest.cc
@@ -33,7 +33,7 @@ namespace chromeos {
class OnlineAttemptTest : public testing::Test {
public:
OnlineAttemptTest()
- : state_(UserContext(), "", "", User::USER_TYPE_REGULAR, false),
+ : state_(UserContext(), false),
attempt_(new OnlineAttempt(&state_, &resolver_)) {
}
@@ -141,8 +141,7 @@ TEST_F(OnlineAttemptTest, HostedLoginRejected) {
// This is how we inject fake URLFetcher objects, with a factory.
MockURLFetcherFactory<HostedFetcher> factory;
- TestAttemptState local_state(UserContext(), "", "",
- User::USER_TYPE_REGULAR, true);
+ TestAttemptState local_state(UserContext(), true);
attempt_.reset(new OnlineAttempt(&local_state, &resolver_));
attempt_->Initiate(&profile);
@@ -164,8 +163,7 @@ TEST_F(OnlineAttemptTest, FullLogin) {
// This is how we inject fake URLFetcher objects, with a factory.
MockURLFetcherFactory<SuccessFetcher> factory;
- TestAttemptState local_state(UserContext(), "", "",
- User::USER_TYPE_REGULAR, true);
+ TestAttemptState local_state(UserContext(), true);
attempt_.reset(new OnlineAttempt(&local_state, &resolver_));
attempt_->Initiate(&profile);

Powered by Google App Engine
This is Rietveld 408576698