| commit | c32295e9208d679e26ca468069373e3ff07be0e8 | |
| author | nkostylev <nkostylev@chromium.org> | |
| Tue, 20 Jan 2015 16:07:41 +0000 (20 08:07 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 20 Jan 2015 16:08:41 +0000 (20 16:08 +0000) | ||
| tree | f2ecedc4b0d75a1cadf81c7e7bc876aca070d13a | treesnapshot (tar.gz zip) |
| parent | 7700b81b49485413154d6f38dd7754ae94bb6567 | commitdiff |
Make sure that user windows are not hidden after session restore.
See previous CL https://codereview.chromium.org/757633003/
which was addressing the case of adding users to the session.
In that case current user windows were hidden immediately. This
should not apply during user session restore process when there's
a different approach (restore all user sessions, then switch to last active user), see https://codereview.chromium.org/594163002
Also sanitize code that looks at the custom flags and does restart.
Fixing bug:
1. Two users in session, second user active.
2. Change flags and initiate restart from chrome://flags
3. End up in the situation when first user is not considered to be properly initialized since were trying to apply session flags.
BUG=443812
Review URL: https://codereview.chromium.org/860823002
Cr-Commit-Position: refs/heads/master@{#312205}
See previous CL https://codereview.chromium.org/757633003/
which was addressing the case of adding users to the session.
In that case current user windows were hidden immediately. This
should not apply during user session restore process when there's
a different approach (restore all user sessions, then switch to last active user), see https://codereview.chromium.org/594163002
Also sanitize code that looks at the custom flags and does restart.
Fixing bug:
1. Two users in session, second user active.
2. Change flags and initiate restart from chrome://flags
3. End up in the situation when first user is not considered to be properly initialized since were trying to apply session flags.
BUG=443812
Review URL: https://codereview.chromium.org/860823002
Cr-Commit-Position: refs/heads/master@{#312205}
| chrome/browser/chromeos/login/session/user_session_manager.cc | diffblobblamehistory | |
| chrome/browser/ui/ash/multi_user/multi_user_window_manager_chromeos.cc | diffblobblamehistory |