Exiting Chrome from the wrench menu should close the User Manager.

Exiting Chrome from the wrench menu should close the User Manager.

The User Manager is not a browser window, so the only way it can stay
open while all browser windows are closed is by having an AutoKeepAlive.
This means, however that if the User Manager is showing, "Exit Chrome" from
the wrench menu will have no affect. The solution is to close the User Manager
first, then exit as before.

BUG=423099
TEST=Start Chrome with --enable-new-avatar-menu. Select "Switch Person" from the
avatar menu. Select "Exit" from the hot dog menu. Chrome should exit.

Committed: https://crrev.com/d710bc8339e2c458f9fd708f0b547d407490a59a
Cr-Commit-Position: refs/heads/master@{#299754}

[noms (inactive), 2014-10-15 15:35:44]

noms@chromium.org changed reviewers:
+ thestig@chromium.org

[noms (inactive), 2014-10-15 15:35:45]

This fixes a bug where trying to exit Chrome from the hot dog menu while the
User Manager was active would not actually exit Chrome. Please take a look.
Thanks!

[Lei Zhang, 2014-10-15 19:07:15]

thestig@chromium.org changed reviewers:
+ mlerman@chromium.org

[Lei Zhang, 2014-10-15 19:07:16]

+mlerman who is probably more familiar with this code.

[noms (inactive), 2014-10-15 19:08:24]

On 2014/10/15 19:07:16, Lei Zhang wrote:
> +mlerman who is probably more familiar with this code.

mlerman is on vacation for two weeks :(
I wrote the User Manager initially, if that's the concern. I don't know much
about application_lifetime.cc, though.

[Lei Zhang, 2014-10-15 19:16:29]

Ok, this makes sense to me, so lgtm.

[noms (inactive), 2014-10-15 20:13:45]

On 2014/10/15 19:16:29, Lei Zhang wrote:
> Ok, this makes sense to me, so lgtm.

Thanks!

[noms (inactive), 2014-10-15 20:13:48]

The CQ bit was checked by noms@chromium.org

[commit-bot: I haz the power, 2014-10-15 20:14:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/658753003/1

[commit-bot: I haz the power, 2014-10-15 20:27:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-15 20:29:19]

Patchset 1 (id:??) landed as
https://crrev.com/d710bc8339e2c458f9fd708f0b547d407490a59a
Cr-Commit-Position: refs/heads/master@{#299754}