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

Unified Diff: chrome/browser/chromeos/login/users/user.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/users/user.h
diff --git a/chrome/browser/chromeos/login/users/user.h b/chrome/browser/chromeos/login/users/user.h
index a3c7be85ae4d4a0d6c8a78f31a862277ccbae883..65ec5e54798f0658b6fdad2c548aaa024263d870 100644
--- a/chrome/browser/chromeos/login/users/user.h
+++ b/chrome/browser/chromeos/login/users/user.h
@@ -36,23 +36,11 @@ class UserContext {
};
UserContext();
- UserContext(const std::string& user_id,
- const std::string& password,
- const std::string& auth_code);
- UserContext(const std::string& user_id,
- const std::string& password,
- const std::string& auth_code,
- const std::string& user_id_hash);
- UserContext(const std::string& user_id,
- const std::string& password,
- const std::string& auth_code,
- const std::string& user_id_hash,
- bool is_using_oauth,
- AuthFlow auth_flow);
+ UserContext(const UserContext& other);
+ explicit UserContext(const std::string& user_id);
~UserContext();
- bool operator==(const UserContext& context) const;
- void CopyFrom(const UserContext& other);
+ bool operator==(const UserContext& context) const;
const std::string& GetUserID() const;
const std::string& GetPassword() const;
@@ -72,6 +60,9 @@ class UserContext {
void SetAuthCode(const std::string& auth_code);
void SetUserIDHash(const std::string& user_id_hash);
void SetIsUsingOAuth(bool is_using_oauth);
+ void SetAuthFlow(AuthFlow auth_flow);
+
+ void ClearSecrets();
private:
std::string user_id_;
« no previous file with comments | « chrome/browser/chromeos/login/signin/oauth2_browsertest.cc ('k') | chrome/browser/chromeos/login/users/user.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698