[ash-md] Cancels overview mode before switching user profiles

[ash-md] Cancels overview mode before switching user profiles

BUG=633580
Committed: https://crrev.com/888ae0b290138fdc505af4a30e75646658b23aa0
Cr-Commit-Position: refs/heads/master@{#414648}

[varkha, 2016-08-24 20:29:25]

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

[commit-bot: I haz the power, 2016-08-24 20:30:44]

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

[varkha, 2016-08-24 20:41:20]

varkha@chromium.org changed reviewers:
+ oshima@chromium.org

[varkha, 2016-08-24 20:41:21]

oshima@, can you please take a look? This seems to be the simplest way.
Thanks!

[commit-bot: I haz the power, 2016-08-24 21:07:06]

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

[commit-bot: I haz the power, 2016-08-24 21:07:07]

Dry run: This issue passed the CQ dry run.

[oshima, 2016-08-24 23:29:12]

can you add a test?

[varkha, 2016-08-25 20:31:44]

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

[varkha, 2016-08-25 20:31:59]

On 2016/08/24 23:29:12, oshima wrote:
> can you add a test?

Done.

[commit-bot: I haz the power, 2016-08-25 20:33:20]

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

[oshima, 2016-08-25 21:29:40]

lgtm

Thanks!

https://codereview.chromium.org/2273103003/diff/20001/chrome/browser/ui/ash/m...
File chrome/browser/ui/ash/multi_user/user_switch_util_unittest.cc (right):

https://codereview.chromium.org/2273103003/diff/20001/chrome/browser/ui/ash/m...
chrome/browser/ui/ash/multi_user/user_switch_util_unittest.cc:94: bool
IsSelecting() { return window_selector_controller()->IsSelecting(); }
nit: const?

[commit-bot: I haz the power, 2016-08-25 21:51:55]

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

[commit-bot: I haz the power, 2016-08-25 21:51:56]

Dry run: This issue passed the CQ dry run.

[varkha, 2016-08-26 03:28:53]

https://codereview.chromium.org/2273103003/diff/20001/chrome/browser/ui/ash/m...
File chrome/browser/ui/ash/multi_user/user_switch_util_unittest.cc (right):

https://codereview.chromium.org/2273103003/diff/20001/chrome/browser/ui/ash/m...
chrome/browser/ui/ash/multi_user/user_switch_util_unittest.cc:94: bool
IsSelecting() { return window_selector_controller()->IsSelecting(); }
On 2016/08/25 21:29:39, oshima wrote:
> nit: const?

Done.

[varkha, 2016-08-26 03:29:00]

The CQ bit was checked by varkha@chromium.org

[varkha, 2016-08-26 03:29:01]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2273103003/#ps40001
(title: "[ash-md] Cancels overview mode before switching user profiles (nit)")

[commit-bot: I haz the power, 2016-08-26 03:29:36]

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, 2016-08-26 04:15:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-26 04:16:52]

Description was changed from

==========
[ash-md] Cancels overview mode before switching user profiles

BUG=633580
==========

to

==========
[ash-md] Cancels overview mode before switching user profiles

BUG=633580

Committed: https://crrev.com/888ae0b290138fdc505af4a30e75646658b23aa0
Cr-Commit-Position: refs/heads/master@{#414648}
==========

[commit-bot: I haz the power, 2016-08-26 04:16:53]

Patchset 3 (id:??) landed as
https://crrev.com/888ae0b290138fdc505af4a30e75646658b23aa0
Cr-Commit-Position: refs/heads/master@{#414648}