[Signin] Fix sign-out flow for unconsented primary account

Fixes two separate issues with the unconsented primary account sign-out
flow within PrimaryAccountMutator and PrimaryAccountManager:
1. Check HasUnconsentedPrimaryAccount in PrimaryAccountMutator so that
   the PrimaryAccountManager::SignOutXYZ is actually invoked even if
   there's no sync consent.
2. Make sure PrimaryAccountMutator calls GoogleSignedOut using a correct
   account info - before this change, it would be invoked with an empty
   account info if there's no sync consent.

Bug: 1130526
Change-Id: I9019faeb602a32a499ae7fd3cab09f5634fc69ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2421739
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809317}

components/signin/internal/identity_manager/primary_account_manager.cc

@@ -351,9 +351,11 @@ void PrimaryAccountManager::OnSignoutDecisionReached(
     return;
   }
 
-  const CoreAccountInfo account_info = GetAuthenticatedAccountInfo();
+  const CoreAccountInfo account_info = primary_account_info();
   client_->GetPrefs()->ClearPref(prefs::kGoogleServicesHostedDomain);
-  SetPrimaryAccountInternal(account_info, /*consented_to_sync=*/false);
+  // Revoke the sync consent.
+  if (IsAuthenticated())
+    SetPrimaryAccountInternal(account_info, /*consented_to_sync=*/false);
 
   // Revoke all tokens before sending signed_out notification, because there
   // may be components that don't listen for token service events when the

components/signin/internal/identity_manager/primary_account_mutator_impl.cc

@@ -78,7 +78,7 @@ bool PrimaryAccountMutatorImpl::ClearPrimaryAccount(
     ClearAccountsAction action,
     signin_metrics::ProfileSignout source_metric,
     signin_metrics::SignoutDelete delete_metric) {
-  if (!primary_account_manager_->IsAuthenticated())
+  if (!primary_account_manager_->HasUnconsentedPrimaryAccount())
     return false;
 
   switch (action) {