Load credentials on desktop even when there is no authenticated account ID.

Load credentials on desktop even when there is no authenticated account ID.

This CL changes the desktop token service to load the credentials for
all accounts even when the SigninManager::authenticated_account_id is
empty.

BUG=733226
Review-Url: https://codereview.chromium.org/2939443004
Cr-Commit-Position: refs/heads/master@{#479693}
Committed: https://chromium.googlesource.com/chromium/src/+/9e8c59957706918a5fbeb97ea4a338271bbc5213

[msarda, 2017-06-13 14:27:01]

The CQ bit was checked by msarda@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-13 14:27:17]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-13 15:34:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-13 15:34:55]

Dry run: This issue passed the CQ dry run.

[msarda, 2017-06-14 09:36:13]

The CQ bit was checked by msarda@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-14 09:36:28]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-14 11:09:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-14 11:09:42]

Dry run: This issue passed the CQ dry run.

[msarda, 2017-06-14 13:15:11]

Description was changed from

==========
Load credentials on desktop even when there is no authenticated account ID.

BUG=
==========

to

==========
Load credentials on desktop even when there is no authenticated account ID.

This CL changes the desktop token service to load the credentials for
all accounts even when the SigninManager::authenticated_account_id is
empty.

BUG=733226
==========

[msarda, 2017-06-14 13:20:01]

msarda@chromium.org changed reviewers:
+ droger@chromium.org, rogerta@chromium.org

[msarda, 2017-06-14 13:20:03]

Please take a look.

[Roger Tawa OOO till Jul 10th, 2017-06-14 19:23:41]

lgtm

FYI, I a wrote CL to do this several years ago which was reverted due to a
regression in startup time for non-signed in users.  You might want to see if
this is still true today or if maybe we are OK with the regression.

[droger, 2017-06-15 11:44:46]

lgtm

[msarda, 2017-06-15 13:07:27]

The CQ bit was checked by msarda@chromium.org

[commit-bot: I haz the power, 2017-06-15 13:07:34]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-15 14:13:47]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497532047514580,
"parent_rev": "71f40c3bd210c7450cbd2e171578c1d950750346", "commit_rev":
"9e8c59957706918a5fbeb97ea4a338271bbc5213"}

[commit-bot: I haz the power, 2017-06-15 14:13:57]

Description was changed from

==========
Load credentials on desktop even when there is no authenticated account ID.

This CL changes the desktop token service to load the credentials for
all accounts even when the SigninManager::authenticated_account_id is
empty.

BUG=733226
==========

to

==========
Load credentials on desktop even when there is no authenticated account ID.

This CL changes the desktop token service to load the credentials for
all accounts even when the SigninManager::authenticated_account_id is
empty.

BUG=733226

Review-Url: https://codereview.chromium.org/2939443004
Cr-Commit-Position: refs/heads/master@{#479693}
Committed:
https://chromium.googlesource.com/chromium/src/+/9e8c59957706918a5fbeb97ea4a3...
==========

[commit-bot: I haz the power, 2017-06-15 14:13:58]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9e8c59957706918a5fbeb97ea4a3...