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

Unified Diff: trunk/src/chrome/browser/chromeos/login/user.h

Issue 137133007: Revert 246674 "Indicate which authentication flow was used in Us..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 11 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: trunk/src/chrome/browser/chromeos/login/user.h
===================================================================
--- trunk/src/chrome/browser/chromeos/login/user.h (revision 246711)
+++ trunk/src/chrome/browser/chromeos/login/user.h (working copy)
@@ -24,16 +24,6 @@
// back once user homedir is mounted. |username_hash| is used to identify
// user homedir mount point.
struct UserContext {
- // The authentication flow used during sign-in.
- enum AuthFlow {
- // Online authentication against GAIA. GAIA did not redirect to a SAML IdP.
- AUTH_FLOW_GAIA_WITHOUT_SAML,
- // Online authentication against GAIA. GAIA redirected to a SAML IdP.
- AUTH_FLOW_GAIA_WITH_SAML,
- // Offline authentication against a cached password.
- AUTH_FLOW_OFFLINE
- };
-
UserContext();
UserContext(const std::string& username,
const std::string& password,
@@ -46,8 +36,7 @@
const std::string& password,
const std::string& auth_code,
const std::string& username_hash,
- bool using_oauth,
- AuthFlow auth_flow);
+ bool using_oauth);
virtual ~UserContext();
bool operator==(const UserContext& context) const;
std::string username;
@@ -55,7 +44,6 @@
std::string auth_code;
std::string username_hash;
bool using_oauth;
- AuthFlow auth_flow;
};
// A class representing information about a previously logged in user.
@@ -162,15 +150,15 @@
// True if image is being loaded from file.
bool image_is_loading() const { return image_is_loading_; }
+ // OAuth token status for this user.
+ OAuthTokenStatus oauth_token_status() const { return oauth_token_status_; }
+
// The displayed user name.
base::string16 display_name() const { return display_name_; }
// The displayed (non-canonical) user email.
virtual std::string display_email() const;
- // OAuth token status for this user.
- OAuthTokenStatus oauth_token_status() const { return oauth_token_status_; }
-
// True if the user's session can be locked (i.e. the user has a password with
// which to unlock the session).
virtual bool can_lock() const;
@@ -223,6 +211,10 @@
// If |is_loading| is |true|, that means user image is being loaded from file.
void SetStubImage(int image_index, bool is_loading);
+ void set_oauth_token_status(OAuthTokenStatus status) {
+ oauth_token_status_ = status;
+ }
+
void set_display_name(const base::string16& display_name) {
display_name_ = display_name;
}
@@ -235,10 +227,6 @@
const UserImage& user_image() const { return user_image_; }
- void set_oauth_token_status(OAuthTokenStatus status) {
- oauth_token_status_ = status;
- }
-
void set_username_hash(const std::string& username_hash) {
username_hash_ = username_hash;
}
« no previous file with comments | « trunk/src/chrome/browser/chromeos/login/screen_locker.cc ('k') | trunk/src/chrome/browser/chromeos/login/user.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698