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

Unified Diff: components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.cc

Issue 1066453002: Refactor CryptAuth component to be more testable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cryptauth_securemessage
Patch Set: rename tests Created 5 years, 8 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: components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.cc
diff --git a/components/proximity_auth/cryptauth/cryptauth_account_token_fetcher.cc b/components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.cc
similarity index 75%
rename from components/proximity_auth/cryptauth/cryptauth_account_token_fetcher.cc
rename to components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.cc
index f02ee681fa35325cdb4223d0f02a02a661dd4f04..294b2ba80f49cfb1602694974a696ff503ca4af2 100644
--- a/components/proximity_auth/cryptauth/cryptauth_account_token_fetcher.cc
+++ b/components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "components/proximity_auth/cryptauth/cryptauth_account_token_fetcher.h"
+#include "components/proximity_auth/cryptauth/cryptauth_access_token_fetcher_impl.h"
namespace proximity_auth {
@@ -17,19 +17,19 @@ OAuth2TokenService::ScopeSet GetScopes() {
} // namespace
-CryptAuthAccountTokenFetcher::CryptAuthAccountTokenFetcher(
+CryptAuthAccessTokenFetcherImpl::CryptAuthAccessTokenFetcherImpl(
OAuth2TokenService* token_service,
const std::string& account_id)
- : OAuth2TokenService::Consumer("cryptauth_account_token_fetcher"),
+ : OAuth2TokenService::Consumer("cryptauth_access_token_fetcher"),
token_service_(token_service),
account_id_(account_id),
fetch_started_(false) {
}
-CryptAuthAccountTokenFetcher::~CryptAuthAccountTokenFetcher() {
+CryptAuthAccessTokenFetcherImpl::~CryptAuthAccessTokenFetcherImpl() {
}
-void CryptAuthAccountTokenFetcher::FetchAccessToken(
+void CryptAuthAccessTokenFetcherImpl::FetchAccessToken(
const AccessTokenCallback& callback) {
if (fetch_started_) {
LOG(WARNING) << "Create an instance for each token fetched. Do not reuse.";
@@ -44,14 +44,14 @@ void CryptAuthAccountTokenFetcher::FetchAccessToken(
token_request_ = token_service_->StartRequest(account_id_, GetScopes(), this);
}
-void CryptAuthAccountTokenFetcher::OnGetTokenSuccess(
+void CryptAuthAccessTokenFetcherImpl::OnGetTokenSuccess(
const OAuth2TokenService::Request* request,
const std::string& access_token,
const base::Time& expiration_time) {
callback_.Run(access_token);
}
-void CryptAuthAccountTokenFetcher::OnGetTokenFailure(
+void CryptAuthAccessTokenFetcherImpl::OnGetTokenFailure(
const OAuth2TokenService::Request* request,
const GoogleServiceAuthError& error) {
callback_.Run(std::string());

Powered by Google App Engine
This is Rietveld 408576698