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

Unified Diff: chrome/browser/chromeos/login/auth/auth_attempt_state.h

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/auth_attempt_state.h
diff --git a/chrome/browser/chromeos/login/auth/auth_attempt_state.h b/chrome/browser/chromeos/login/auth/auth_attempt_state.h
index de9bb03fc36d5ebaf41bdd879ebc82856f81431c..090cc36a22ca2b19417953c4f65582c09f2089a8 100644
--- a/chrome/browser/chromeos/login/auth/auth_attempt_state.h
+++ b/chrome/browser/chromeos/login/auth/auth_attempt_state.h
@@ -15,24 +15,16 @@
namespace chromeos {
-// Tracks the state associated with a single attempt to log in to chromium os.
+// Tracks the state associated with a single attempt to log in to chromium OS.
// Enforces that methods are only called on the UI thread.
-
class AuthAttemptState {
public:
// Used to initialize for a login attempt.
AuthAttemptState(const UserContext& user_context,
- const std::string& login_token,
- const std::string& login_captcha,
- const User::UserType user_type,
- const bool user_is_new);
-
- // Used to initialize for a externally authenticated login.
- AuthAttemptState(const UserContext& user_context,
- const bool user_is_new);
-
- // Used to initialize for a screen unlock attempt.
- AuthAttemptState(const std::string& username, const std::string& password);
+ User::UserType user_type,
+ bool unlock,
+ bool online_complete,
+ bool user_is_new);
virtual ~AuthAttemptState();

Powered by Google App Engine
This is Rietveld 408576698