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

Unified Diff: chrome/browser/signin/dice_response_handler.cc

Issue 2942193002: [signin] Generate OAuth token on Dice Signin responses (Closed)
Patch Set: Review comments Created 3 years, 6 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
« no previous file with comments | « chrome/browser/signin/dice_response_handler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/signin/dice_response_handler.cc
diff --git a/chrome/browser/signin/dice_response_handler.cc b/chrome/browser/signin/dice_response_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2912fc374185fe650fa4c7deb7f91a9397c0e9d9
--- /dev/null
+++ b/chrome/browser/signin/dice_response_handler.cc
@@ -0,0 +1,148 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/signin/dice_response_handler.h"
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/signin/account_tracker_service_factory.h"
+#include "chrome/browser/signin/chrome_signin_client_factory.h"
+#include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/keyed_service/content/browser_context_keyed_service_factory.h"
+#include "components/signin/core/browser/account_tracker_service.h"
+#include "components/signin/core/browser/profile_oauth2_token_service.h"
+#include "components/signin/core/browser/signin_client.h"
+#include "components/signin/core/browser/signin_header_helper.h"
+#include "components/signin/core/common/profile_management_switches.h"
+#include "google_apis/gaia/gaia_auth_fetcher.h"
+#include "google_apis/gaia/gaia_constants.h"
+#include "google_apis/gaia/google_service_auth_error.h"
+
+namespace {
+
+class DiceResponseHandlerFactory : public BrowserContextKeyedServiceFactory {
+ public:
+ // Returns an instance of the factory singleton.
+ static DiceResponseHandlerFactory* GetInstance() {
+ return base::Singleton<DiceResponseHandlerFactory>::get();
+ }
+
+ static DiceResponseHandler* GetForProfile(Profile* profile) {
+ return static_cast<DiceResponseHandler*>(
+ GetInstance()->GetServiceForBrowserContext(profile, true));
+ }
+
+ private:
+ friend struct base::DefaultSingletonTraits<DiceResponseHandlerFactory>;
+
+ DiceResponseHandlerFactory()
+ : BrowserContextKeyedServiceFactory(
+ "DiceResponseHandler",
+ BrowserContextDependencyManager::GetInstance()) {
+ DependsOn(AccountTrackerServiceFactory::GetInstance());
+ DependsOn(ChromeSigninClientFactory::GetInstance());
+ DependsOn(ProfileOAuth2TokenServiceFactory::GetInstance());
+ }
+
+ ~DiceResponseHandlerFactory() override {}
+
+ // BrowserContextKeyedServiceFactory:
+ KeyedService* BuildServiceInstanceFor(
+ content::BrowserContext* context) const override {
+ if (context->IsOffTheRecord())
+ return nullptr;
+
+ Profile* profile = static_cast<Profile*>(context);
+ return new DiceResponseHandler(
+ ChromeSigninClientFactory::GetForProfile(profile),
+ ProfileOAuth2TokenServiceFactory::GetForProfile(profile),
+ AccountTrackerServiceFactory::GetForProfile(profile));
+ }
+};
+
+} // namespace
+
+// static
+DiceResponseHandler* DiceResponseHandler::GetForProfile(Profile* profile) {
+ return DiceResponseHandlerFactory::GetForProfile(profile);
+}
+
+DiceResponseHandler::DiceResponseHandler(
+ SigninClient* signin_client,
+ ProfileOAuth2TokenService* profile_oauth2_token_service,
+ AccountTrackerService* account_tracker_service)
+ : signin_client_(signin_client),
+ token_service_(profile_oauth2_token_service),
+ account_tracker_service_(account_tracker_service) {
+ DCHECK(signin_client_);
+ DCHECK(token_service_);
+ DCHECK(account_tracker_service_);
+}
+
+DiceResponseHandler::~DiceResponseHandler() {}
+
+void DiceResponseHandler::ProcessDiceHeader(
+ const signin::DiceResponseParams& dice_params) {
+ DCHECK_EQ(switches::AccountConsistencyMethod::kDice,
+ switches::GetAccountConsistencyMethod());
+
+ switch (dice_params.user_intention) {
+ case signin::DiceAction::SIGNIN:
+ ProcessDiceSigninHeader(dice_params.obfuscated_gaia_id, dice_params.email,
+ dice_params.authorization_code);
+ return;
+ case signin::DiceAction::SIGNOUT:
+ case signin::DiceAction::SINGLE_SESSION_SIGNOUT:
+ LOG(ERROR) << "Signout through Dice is not implemented.";
+ return;
+ case signin::DiceAction::NONE:
+ NOTREACHED() << "Invalid Dice response parameters.";
+ return;
+ }
+
+ NOTREACHED();
+ return;
+}
+
+void DiceResponseHandler::ProcessDiceSigninHeader(
+ const std::string& gaia_id,
+ const std::string& email,
+ const std::string& authorization_code) {
+ DCHECK(!gaia_id.empty());
+ DCHECK(!email.empty());
+ DCHECK(!authorization_code.empty());
+ DCHECK(!gaia_auth_fetcher_);
+ DCHECK(gaia_id_.empty());
+ DCHECK(email_.empty());
+ gaia_id_ = gaia_id;
+ email_ = email;
+ gaia_auth_fetcher_ = signin_client_->CreateGaiaAuthFetcher(
+ this, GaiaConstants::kChromeSource,
+ signin_client_->GetURLRequestContext());
+ gaia_auth_fetcher_->StartAuthCodeForOAuth2TokenExchange(authorization_code);
+
+ // TODO(droger): The token exchange must complete quickly or be cancelled. Add
+ // a timeout logic.
+}
+
+void DiceResponseHandler::OnClientOAuthSuccess(
+ const ClientOAuthResult& result) {
+ std::string account_id =
+ account_tracker_service_->SeedAccountInfo(gaia_id_, email_);
+ VLOG(1) << "Dice OAuth success for account: " << account_id;
+ token_service_->UpdateCredentials(account_id, result.refresh_token);
+ gaia_id_.clear();
+ email_.clear();
+ gaia_auth_fetcher_.reset();
+}
+
+void DiceResponseHandler::OnClientOAuthFailure(
+ const GoogleServiceAuthError& error) {
+ // TODO(droger): Handle authentication errors.
+ VLOG(1) << "Dice OAuth failed with error: " << error.ToString();
+ gaia_id_.clear();
+ email_.clear();
+ gaia_auth_fetcher_.reset();
+}
« no previous file with comments | « chrome/browser/signin/dice_response_handler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698