Commit a85032e3 authored by Sergio Villar Senin's avatar Sergio Villar Senin Committed by Commit Bot

Convert IOSChromeSigninStatusMetricsProviderDelegate to IdentityManager

It used to call SiginManager::IsAuthenticated(). Replaced that call by
IdentityManager::HasPrimaryAccount().

Bug: 890822
Change-Id: If6e24c69db6f4634a5f96110ad9ca590b6d74d45
Reviewed-on: https://chromium-review.googlesource.com/c/1329677Reviewed-by: default avatarMihai Sardarescu <msarda@chromium.org>
Reviewed-by: default avatarColin Blundell <blundell@chromium.org>
Commit-Queue: Sergio Villar <svillar@igalia.com>
Cr-Commit-Position: refs/heads/master@{#608343}
parent 8532e551
...@@ -9,7 +9,9 @@ ...@@ -9,7 +9,9 @@
#include "ios/chrome/browser/application_context.h" #include "ios/chrome/browser/application_context.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state.h" #include "ios/chrome/browser/browser_state/chrome_browser_state.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state_manager.h" #include "ios/chrome/browser/browser_state/chrome_browser_state_manager.h"
#include "ios/chrome/browser/signin/identity_manager_factory.h"
#include "ios/chrome/browser/signin/signin_manager_factory.h" #include "ios/chrome/browser/signin/signin_manager_factory.h"
#include "services/identity/public/cpp/identity_manager.h"
IOSChromeSigninStatusMetricsProviderDelegate:: IOSChromeSigninStatusMetricsProviderDelegate::
IOSChromeSigninStatusMetricsProviderDelegate() {} IOSChromeSigninStatusMetricsProviderDelegate() {}
...@@ -36,9 +38,9 @@ IOSChromeSigninStatusMetricsProviderDelegate::GetStatusOfAllAccounts() { ...@@ -36,9 +38,9 @@ IOSChromeSigninStatusMetricsProviderDelegate::GetStatusOfAllAccounts() {
accounts_status.num_opened_accounts = accounts_status.num_accounts; accounts_status.num_opened_accounts = accounts_status.num_accounts;
for (ios::ChromeBrowserState* browser_state : browser_state_list) { for (ios::ChromeBrowserState* browser_state : browser_state_list) {
SigninManager* manager = ios::SigninManagerFactory::GetForBrowserState( auto* manager = IdentityManagerFactory::GetForBrowserState(
browser_state->GetOriginalChromeBrowserState()); browser_state->GetOriginalChromeBrowserState());
if (manager && manager->IsAuthenticated()) if (manager && manager->HasPrimaryAccount())
accounts_status.num_signed_in_accounts++; accounts_status.num_signed_in_accounts++;
} }
......
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