CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351
Committed: https://crrev.com/bfcf715cb1a1dca220fdbbc662c4f875a2dbc26b
Cr-Commit-Position: refs/heads/master@{#441242}

[Evan Stade, 2017-01-03 22:17:29]

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

[Evan Stade, 2017-01-03 22:23:44]

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

[commit-bot: I haz the power, 2017-01-03 22:24:21]

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

[varkha, 2017-01-03 22:38:52]

lgtm.

https://codereview.chromium.org/2612633002/diff/1/ash/common/wm/window_cycle_...
File ash/common/wm/window_cycle_list.cc (left):

https://codereview.chromium.org/2612633002/diff/1/ash/common/wm/window_cycle_...
ash/common/wm/window_cycle_list.cc:428: ScopedShowWindow::ScopedShowWindow()
Since you are removing this pre-MD path, maybe update the CL description to
mention it.

[Evan Stade, 2017-01-03 22:41:36]

Description was changed from

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

BUG=648351
==========

to

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351
==========

[Evan Stade, 2017-01-03 22:41:47]

https://codereview.chromium.org/2612633002/diff/1/ash/common/wm/window_cycle_...
File ash/common/wm/window_cycle_list.cc (left):

https://codereview.chromium.org/2612633002/diff/1/ash/common/wm/window_cycle_...
ash/common/wm/window_cycle_list.cc:428: ScopedShowWindow::ScopedShowWindow()
On 2017/01/03 22:38:51, varkha wrote:
> Since you are removing this pre-MD path, maybe update the CL description to
> mention it.

Done.

[commit-bot: I haz the power, 2017-01-03 22:52:42]

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

[commit-bot: I haz the power, 2017-01-03 22:52:43]

Dry run: This issue passed the CQ dry run.

[Evan Stade, 2017-01-03 23:09:48]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2017-01-03 23:10:16]

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-01-03 23:14:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483484988651530, "parent_rev":
"4135f4f05c163872d3ec68f9e471c6bca0dfc35b", "commit_rev":
"2c5414cd19702d9dddc21d3e894ee2a6d03085b3"}

[commit-bot: I haz the power, 2017-01-03 23:15:03]

Description was changed from

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351
==========

to

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351

Review-Url: https://codereview.chromium.org/2612633002
==========

[commit-bot: I haz the power, 2017-01-03 23:15:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-03 23:18:54]

Description was changed from

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351

Review-Url: https://codereview.chromium.org/2612633002
==========

to

==========
CrOS Alt+Tab UI: no-op when UI is aborted (instead of switching window).

Also, remove the old pre-MD code for window switching & activation.

BUG=648351

Committed: https://crrev.com/bfcf715cb1a1dca220fdbbc662c4f875a2dbc26b
Cr-Commit-Position: refs/heads/master@{#441242}
==========

[commit-bot: I haz the power, 2017-01-03 23:18:54]

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