commit c735d674a18dc4766ae616b04b057e7435b9b80b
author Marc Treib <treib@chromium.org> Mon Mar 18 16:34:15 2019
committer Commit Bot <commit-bot@chromium.org> Mon Mar 18 16:34:15 2019
tree a1eda2895cf70f68cc49c6781c094c900a007183
parent 1f6ab76b7085ef0726d63a5a40ffd6e8d1b6ef8d

sync_ui_util cleanup: Break apart unrelated cases

Before this CL, there was a condition
"if (disabled_by_policy || !requested || first_setup_complete)"
which didn't make a lot of sense.
This CL breaks it apart by pulling out the disabled_by_policy and
!requested cases.
As one potential small behavior difference, these two cases will now have
priority over the "passphrase required" case. A passphrase should never
be required if Sync is disabled by policy or disabled by the user, so
this should make no practical difference.

Bug: 911153
Change-Id: I000192b4f17578cf1d1ff714e7f4e9ef4c752131
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1528612
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#641641}