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

Unified Diff: chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc

Issue 2864433003: [ARC] Add browser test of ArcAuthService (Closed)
Patch Set: Response to review Created 3 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/arc/auth/arc_background_auth_code_fetcher.cc
diff --git a/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc b/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
index ed80b1540eebe03abd14cc93bcb5cff9b1c311fc..8472aaaea079e0497030e4edb84fb10df0dadb29 100644
--- a/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
+++ b/chrome/browser/chromeos/arc/auth/arc_background_auth_code_fetcher.cc
@@ -12,12 +12,9 @@
#include "base/json/json_writer.h"
#include "base/logging.h"
#include "base/values.h"
-#include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
-#include "chrome/browser/signin/signin_manager_factory.h"
#include "chrome/browser/ui/ash/multi_user/multi_user_util.h"
#include "components/signin/core/account_id/account_id.h"
#include "components/signin/core/browser/profile_oauth2_token_service.h"
-#include "components/signin/core/browser/signin_manager_base.h"
#include "components/user_manager/known_user.h"
#include "content/public/browser/browser_context.h"
#include "content/public/common/url_constants.h"
@@ -57,6 +54,11 @@ ArcBackgroundAuthCodeFetcher::ArcBackgroundAuthCodeFetcher(
ArcBackgroundAuthCodeFetcher::~ArcBackgroundAuthCodeFetcher() = default;
+// static
+std::string ArcBackgroundAuthCodeFetcher::GetTokenExchangeEndpointForTesting() {
+ return kEndPoint;
khmel 2017/05/12 15:52:28 nit: probably better to expose this via header
blundell 2017/05/15 08:32:32 Done.
+}
+
void ArcBackgroundAuthCodeFetcher::Fetch(const FetchCallback& callback) {
DCHECK(callback_.is_null());
callback_ = callback;

Powered by Google App Engine
This is Rietveld 408576698