Make sure primary root window is deleted last.

Make sure primary root window is deleted last.

BUG=403380
R=mukai@chromium.org

Committed: https://crrev.com/892f21f9f31a9b7e2bd23430f90c74d1a9dc33f0
Cr-Commit-Position: refs/heads/master@{#292759}

[oshima, 2014-08-30 00:50:36]

https://codereview.chromium.org/520273002/diff/20001/ash/display/display_cont...
File ash/display/display_controller.cc (left):

https://codereview.chromium.org/520273002/diff/20001/ash/display/display_cont...
ash/display/display_controller.cc:270: // from the last so that the primary root
window gets deleted last.
This comment was wrong, although deleting primary display will
reassign the primary, so it shouldn't cause a crash.

This is a speculative fix as I couldn't find a repro step/path.

https://codereview.chromium.org/520273002/diff/20001/ash/display/display_cont...
File ash/display/display_controller.cc (right):

https://codereview.chromium.org/520273002/diff/20001/ash/display/display_cont...
ash/display/display_controller.cc:339: CHECK_EQ(1u,
window_tree_hosts_.count(id));
I changed to CHECK to catch the error early in release build.

[Jun Mukai, 2014-08-30 01:01:15]

lgtm

[oshima, 2014-08-30 01:01:55]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-08-30 01:03:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/520273002/20001

[commit-bot: I haz the power, 2014-08-30 02:11:37]

Committed patchset #1 (id:20001) as cabeae2d354655fce26f8074f521d6c39fb1bdb2

[commit-bot: I haz the power, 2014-09-10 03:13:20]

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