0

/OAuthLogin requests should use ChannelID.

R=mattm@chromium.org
BUG=315301

Review URL: https://codereview.chromium.org/70633004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235076 0039d316-1c4b-4281-b951-d872f2087c98
This commit is contained in:
thaidn@google.com
2013-11-14 06:55:57 +00:00
parent cb20fb3ec5
commit 1406864f87
3 changed files with 10 additions and 11 deletions
chrome/browser/chromeos/login
google_apis/gaia

@@ -97,11 +97,11 @@ void OAuth2LoginVerifier::StartFetchingOAuthLoginAccessToken(Profile* profile) {
void OAuth2LoginVerifier::StartOAuthLoginForUberToken() {
// No service will fetch us uber auth token.
gaia_system_fetcher_.reset(
gaia_fetcher_.reset(
new GaiaAuthFetcher(this,
std::string(GaiaConstants::kChromeOSSource),
system_request_context_.get()));
gaia_system_fetcher_->StartTokenFetchForUberAuthExchange(access_token_);
user_request_context_.get()));
gaia_fetcher_->StartTokenFetchForUberAuthExchange(access_token_);
}
@@ -127,11 +127,11 @@ void OAuth2LoginVerifier::OnUberAuthTokenFailure(
void OAuth2LoginVerifier::StartOAuthLoginForGaiaCredentials() {
// No service will fetch us uber auth token.
gaia_system_fetcher_.reset(
gaia_fetcher_.reset(
new GaiaAuthFetcher(this,
std::string(GaiaConstants::kChromeOSSource),
system_request_context_.get()));
gaia_system_fetcher_->StartOAuthLogin(access_token_, EmptyString());
user_request_context_.get()));
gaia_fetcher_->StartOAuthLogin(access_token_, EmptyString());
}
void OAuth2LoginVerifier::OnClientLoginSuccess(

@@ -99,7 +99,6 @@ class OAuth2LoginVerifier : public base::SupportsWeakPtr<OAuth2LoginVerifier>,
OAuth2LoginVerifier::Delegate* delegate_;
scoped_refptr<net::URLRequestContextGetter> system_request_context_;
scoped_refptr<net::URLRequestContextGetter> user_request_context_;
scoped_ptr<GaiaAuthFetcher> gaia_system_fetcher_;
scoped_ptr<GaiaAuthFetcher> gaia_fetcher_;
std::string access_token_;
std::string gaia_token_;

@@ -220,8 +220,8 @@ net::URLFetcher* GaiaAuthFetcher::CreateGaiaFetcher(
// The Gaia token exchange requests do not require any cookie-based
// identification as part of requests. We suppress sending any cookies to
// maintain a separation between the user's browsing and Chrome's internal
// services. Where such mixing is desired (MergeSession), it will be done
// explicitly.
// services. Where such mixing is desired (MergeSession or OAuthLogin), it
// will be done explicitly.
to_return->SetLoadFlags(load_flags);
// Fetchers are sometimes cancelled because a network change was detected,
@@ -637,7 +637,7 @@ void GaiaAuthFetcher::StartTokenFetchForUberAuthExchange(
std::string(),
authentication_header,
uberauth_token_gurl_,
kLoadFlagsIgnoreCookies,
net::LOAD_NORMAL,
this));
fetch_pending_ = true;
fetcher_->Start();
@@ -654,7 +654,7 @@ void GaiaAuthFetcher::StartOAuthLogin(const std::string& access_token,
request_body_,
authentication_header,
oauth_login_gurl_,
kLoadFlagsIgnoreCookies,
net::LOAD_NORMAL,
this));
fetch_pending_ = true;
fetcher_->Start();