Only show UserManager and sign in dialog after profile deleting if fallback profile is locked, when…

Only show UserManager and sign in dialog after profile deleting if fallback profile is locked, when force-sign-in policy is enabled.

BUG=642059
Review-Url: https://codereview.chromium.org/2715543004
Cr-Commit-Position: refs/heads/master@{#453369}
Committed: https://chromium.googlesource.com/chromium/src/+/d00656ece1ae3378d3206cb5aca23d4566158b6d

[zmin, 2017-02-23 23:46:55]

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

[commit-bot: I haz the power, 2017-02-23 23:47:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[zmin, 2017-02-23 23:47:50]

zmin@chromium.org changed reviewers:
+ tommycli@chromium.org

[zmin, 2017-02-23 23:47:51]

Hi tommycli,

Can you review this CL please?

Owen

[commit-bot: I haz the power, 2017-02-24 02:54:20]

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

[commit-bot: I haz the power, 2017-02-24 02:54:21]

Dry run: This issue passed the CQ dry run.

[tommycli, 2017-02-27 21:38:34]

lgtm if the following understanding is correct:

We only want to show the user manager if the fallback profile requires sign-in
to use. Otherwise, just allow the user to use the fallback profile without
signing in...

[zmin, 2017-02-27 21:43:06]

On 2017/02/27 21:38:34, tommycli wrote:
> lgtm if the following understanding is correct:
> 
> We only want to show the user manager if the fallback profile requires sign-in
> to use. Otherwise, just allow the user to use the fallback profile without
> signing in...

Yes, that's correct. Thanks a lot for the review.

[zmin, 2017-02-27 21:43:09]

The CQ bit was checked by zmin@chromium.org

[commit-bot: I haz the power, 2017-02-27 21:43:44]

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-02-27 22:50:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488231789653490, "parent_rev":
"19c0e98f042d6d357e07c101911a724f0086a394", "commit_rev":
"d00656ece1ae3378d3206cb5aca23d4566158b6d"}

[commit-bot: I haz the power, 2017-02-27 22:50:51]

Description was changed from

==========
Only show UserManager and sign in dialog after profile deleting if fallback
profile is locked, when force-sign-in policy is enabled.

BUG=642059
==========

to

==========
Only show UserManager and sign in dialog after profile deleting if fallback
profile is locked, when force-sign-in policy is enabled.

BUG=642059

Review-Url: https://codereview.chromium.org/2715543004
Cr-Commit-Position: refs/heads/master@{#453369}
Committed:
https://chromium.googlesource.com/chromium/src/+/d00656ece1ae3378d3206cb5aca2...
==========

[commit-bot: I haz the power, 2017-02-27 22:50:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d00656ece1ae3378d3206cb5aca2...