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

Unified Diff: chrome/browser/sync/signin_manager.cc

Issue 7574009: Added OAuth support to TokenService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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/sync/signin_manager.cc
diff --git a/chrome/browser/sync/signin_manager.cc b/chrome/browser/sync/signin_manager.cc
index bdccf86705d08f8eb5e299052d8cadc530ee0202..2a697f5c1b4428bb810894b84f16d988c3c0631f 100644
--- a/chrome/browser/sync/signin_manager.cc
+++ b/chrome/browser/sync/signin_manager.cc
@@ -18,8 +18,6 @@
const char kGetInfoEmailKey[] = "email";
-const char kSyncOAuth2Scope[] = "https://www.googleapis.com/auth/chromesync";
-
SigninManager::SigninManager()
: profile_(NULL), had_two_factor_error_(false) {}
@@ -84,8 +82,7 @@ void SigninManager::StartOAuthSignIn() {
oauth_login_.reset(new GaiaOAuthFetcher(this,
profile_->GetRequestContext(),
profile_,
- GaiaConstants::kSyncService,
- kSyncOAuth2Scope));
+ GaiaConstants::kSyncServiceOAuth));
oauth_login_->StartGetOAuthToken();
}
@@ -245,6 +242,7 @@ void SigninManager::OnOAuthWrapBridgeSuccess(const std::string& service_name,
}
void SigninManager::OnOAuthWrapBridgeFailure(
+ const std::string& service_scope,
const GoogleServiceAuthError& error) {
VLOG(1) << "SigninManager::OnOAuthWrapBridgeFailure";
}

Powered by Google App Engine
This is Rietveld 408576698