Commit 4c6f6e21 authored by Antonio Gomes's avatar Antonio Gomes Committed by Commit Bot

[s13n] Convert c/b/profiles/profile_manager.cc to IdentityManager

BUG=903749

Change-Id: I1c904832eb6dcd8de0e670af8ff4e9221d487d69
Reviewed-on: https://chromium-review.googlesource.com/c/1329081Reviewed-by: default avatarMihai Sardarescu <msarda@chromium.org>
Reviewed-by: default avatarColin Blundell <blundell@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#607571}
parent e26c11f0
...@@ -54,9 +54,8 @@ ...@@ -54,9 +54,8 @@
#include "chrome/browser/sessions/session_service_factory.h" #include "chrome/browser/sessions/session_service_factory.h"
#include "chrome/browser/signin/account_fetcher_service_factory.h" #include "chrome/browser/signin/account_fetcher_service_factory.h"
#include "chrome/browser/signin/account_reconcilor_factory.h" #include "chrome/browser/signin/account_reconcilor_factory.h"
#include "chrome/browser/signin/account_tracker_service_factory.h"
#include "chrome/browser/signin/gaia_cookie_manager_service_factory.h" #include "chrome/browser/signin/gaia_cookie_manager_service_factory.h"
#include "chrome/browser/signin/signin_manager_factory.h" #include "chrome/browser/signin/identity_manager_factory.h"
#include "chrome/browser/signin/signin_util.h" #include "chrome/browser/signin/signin_util.h"
#include "chrome/browser/sync/profile_sync_service_factory.h" #include "chrome/browser/sync/profile_sync_service_factory.h"
#include "chrome/browser/ui/browser.h" #include "chrome/browser/ui/browser.h"
...@@ -86,9 +85,7 @@ ...@@ -86,9 +85,7 @@
#include "components/prefs/scoped_user_pref_update.h" #include "components/prefs/scoped_user_pref_update.h"
#include "components/search_engines/default_search_manager.h" #include "components/search_engines/default_search_manager.h"
#include "components/signin/core/browser/account_fetcher_service.h" #include "components/signin/core/browser/account_fetcher_service.h"
#include "components/signin/core/browser/account_tracker_service.h"
#include "components/signin/core/browser/gaia_cookie_manager_service.h" #include "components/signin/core/browser/gaia_cookie_manager_service.h"
#include "components/signin/core/browser/signin_manager.h"
#include "components/signin/core/browser/signin_pref_names.h" #include "components/signin/core/browser/signin_pref_names.h"
#include "components/sync/base/stop_source.h" #include "components/sync/base/stop_source.h"
#include "content/public/browser/browser_task_traits.h" #include "content/public/browser/browser_task_traits.h"
...@@ -100,6 +97,7 @@ ...@@ -100,6 +97,7 @@
#include "net/url_request/url_request_context.h" #include "net/url_request/url_request_context.h"
#include "net/url_request/url_request_context_getter.h" #include "net/url_request/url_request_context_getter.h"
#include "net/url_request/url_request_job.h" #include "net/url_request/url_request_job.h"
#include "services/identity/public/cpp/identity_manager.h"
#include "ui/base/l10n/l10n_util.h" #include "ui/base/l10n/l10n_util.h"
#if BUILDFLAG(ENABLE_EXTENSIONS) #if BUILDFLAG(ENABLE_EXTENSIONS)
...@@ -350,14 +348,6 @@ bool IsProfileEphemeral(ProfileAttributesStorage* storage, ...@@ -350,14 +348,6 @@ bool IsProfileEphemeral(ProfileAttributesStorage* storage,
} }
#endif #endif
#if !defined(OS_ANDROID) && !defined(OS_CHROMEOS)
void SignOut(SigninManager* signin_manager) {
signin_manager->SignOutAndRemoveAllAccounts(
signin_metrics::AUTHENTICATION_FAILED_WITH_FORCE_SIGNIN,
signin_metrics::SignoutDelete::IGNORE_METRIC);
}
#endif
} // namespace } // namespace
ProfileManager::ProfileManager(const base::FilePath& user_data_dir) ProfileManager::ProfileManager(const base::FilePath& user_data_dir)
...@@ -1657,13 +1647,9 @@ void ProfileManager::AddProfileToStorage(Profile* profile) { ...@@ -1657,13 +1647,9 @@ void ProfileManager::AddProfileToStorage(Profile* profile) {
return; return;
} }
identity::IdentityManager* identity_manager =
SigninManagerBase* signin_manager = IdentityManagerFactory::GetForProfile(profile);
SigninManagerFactory::GetForProfile(profile); AccountInfo account_info = identity_manager->GetPrimaryAccountInfo();
AccountTrackerService* account_tracker =
AccountTrackerServiceFactory::GetForProfile(profile);
AccountInfo account_info = account_tracker->GetAccountInfo(
signin_manager->GetAuthenticatedAccountId());
base::string16 username = base::UTF8ToUTF16(account_info.email); base::string16 username = base::UTF8ToUTF16(account_info.email);
ProfileAttributesStorage& storage = GetProfileAttributesStorage(); ProfileAttributesStorage& storage = GetProfileAttributesStorage();
...@@ -1677,7 +1663,7 @@ void ProfileManager::AddProfileToStorage(Profile* profile) { ...@@ -1677,7 +1663,7 @@ void ProfileManager::AddProfileToStorage(Profile* profile) {
#if !defined(OS_ANDROID) && !defined(OS_CHROMEOS) #if !defined(OS_ANDROID) && !defined(OS_CHROMEOS)
bool was_authenticated_status = entry->IsAuthenticated(); bool was_authenticated_status = entry->IsAuthenticated();
#endif #endif
// The ProfileAttributesStorage's info must match the Signin Manager. // The ProfileAttributesStorage's info must match the Identity Manager.
entry->SetAuthInfo(account_info.gaia, username); entry->SetAuthInfo(account_info.gaia, username);
#if !defined(OS_ANDROID) && !defined(OS_CHROMEOS) #if !defined(OS_ANDROID) && !defined(OS_CHROMEOS)
// Sign out if force-sign-in policy is enabled and profile is not signed // Sign out if force-sign-in policy is enabled and profile is not signed
...@@ -1686,8 +1672,12 @@ void ProfileManager::AddProfileToStorage(Profile* profile) { ...@@ -1686,8 +1672,12 @@ void ProfileManager::AddProfileToStorage(Profile* profile) {
!entry->IsAuthenticated()) { !entry->IsAuthenticated()) {
base::PostTaskWithTraits( base::PostTaskWithTraits(
FROM_HERE, {BrowserThread::UI}, FROM_HERE, {BrowserThread::UI},
base::BindOnce(&SignOut, SigninManager::FromSigninManagerBase( base::BindOnce(
signin_manager))); &identity::IdentityManager::ClearPrimaryAccount,
base::Unretained(identity_manager),
identity::IdentityManager::ClearAccountTokensAction::kRemoveAll,
signin_metrics::AUTHENTICATION_FAILED_WITH_FORCE_SIGNIN,
signin_metrics::SignoutDelete::IGNORE_METRIC));
} }
#endif #endif
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment