Commit c46496d3 authored by courage@chromium.org's avatar courage@chromium.org

Fix chrome.identity.onSignInChanged regression for sign-outs

This change fixes a regression from r270325, allowing sign-out events
to fire when the --enable-new-profile-management flag is absent, which
is the default.

The previous change added a check to filter out events related to
non-primary accounts, but the primary account ID is not available
during sign-out. Removing the filter fixes the regression.

BUG=376421

Review URL: https://codereview.chromium.org/346913003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278762 0039d316-1c4b-4281-b951-d872f2087c98
parent 9ac6afc5
...@@ -219,12 +219,6 @@ void IdentityAPI::OnAccountRemoved(const AccountIds& ids) {} ...@@ -219,12 +219,6 @@ void IdentityAPI::OnAccountRemoved(const AccountIds& ids) {}
void IdentityAPI::OnAccountSignInChanged(const AccountIds& ids, void IdentityAPI::OnAccountSignInChanged(const AccountIds& ids,
bool is_signed_in) { bool is_signed_in) {
const std::string primary_account_id = GetPrimaryAccountId(browser_context_);
if (primary_account_id != ids.account_key &&
!switches::IsExtensionsMultiAccount()) {
return;
}
api::identity::AccountInfo account_info; api::identity::AccountInfo account_info;
account_info.id = ids.gaia; account_info.id = ids.gaia;
......
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