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

Unified Diff: chrome/browser/chromeos/login/parallel_authenticator.h

Issue 8761016: Shaving parallel authenticator yak to remove unnecessary dependency on this class from OAuth spec... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years 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/parallel_authenticator.h
===================================================================
--- chrome/browser/chromeos/login/parallel_authenticator.h (revision 112673)
+++ chrome/browser/chromeos/login/parallel_authenticator.h (working copy)
@@ -147,11 +147,6 @@
const std::string& password,
const std::string& login_token,
const std::string& login_captcha) OVERRIDE;
- virtual void VerifyOAuth1AccessToken(const std::string& oauth1_access_token,
- const std::string& oauth1_secret) OVERRIDE;
- virtual std::string EncryptToken(const std::string& token) OVERRIDE;
- virtual std::string DecryptToken(const std::string& encrypted_token) OVERRIDE;
-
// AuthAttemptStateResolver overrides.
// Attempts to make a decision and call back |consumer_| based on
// the state we have gathered at the time of call. If a decision
@@ -239,12 +234,6 @@
// Records OAuth1 access token verification failure for |user_account|.
void RecordOAuthCheckFailure(const std::string& user_account);
- // Stores a hash of |password|, salted with the ascii of |system_salt_|.
- std::string HashPassword(const std::string& password);
-
- // Returns the ascii encoding of the system salt.
- std::string SaltAsAscii();
-
// Signal login completion status for cases when a new user is added via
// an external authentication provider (i.e. GAIA extension).
void ResolveLoginCompletionStatus();
@@ -273,10 +262,6 @@
scoped_refptr<CryptohomeOp> guest_mounter_;
scoped_refptr<CryptohomeOp> key_checker_;
- std::string ascii_hash_;
- chromeos::CryptohomeBlob system_salt_;
- scoped_ptr<crypto::SymmetricKey> supplemental_user_key_;
-
// When the user has changed her password, but gives us the old one, we will
// be able to mount her cryptohome, but online authentication will fail.
// This allows us to present the same behavior to the caller, regardless
@@ -294,7 +279,6 @@
friend class ResolveChecker;
friend class ParallelAuthenticatorTest;
- FRIEND_TEST_ALL_PREFIXES(ParallelAuthenticatorTest, SaltToAscii);
FRIEND_TEST_ALL_PREFIXES(ParallelAuthenticatorTest, ReadLocalaccount);
FRIEND_TEST_ALL_PREFIXES(ParallelAuthenticatorTest,
ReadLocalaccountTrailingWS);
« no previous file with comments | « chrome/browser/chromeos/login/mock_authenticator.cc ('k') | chrome/browser/chromeos/login/parallel_authenticator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698