mus: Do not notify clients when the roots are reordered.

mus: Do not notify clients when the roots are reordered.

BUG=none

Committed: https://crrev.com/04a9cf00d60912dde3e6fe4caa66b205d14acb68
Cr-Commit-Position: refs/heads/master@{#369098}

[sadrul, 2016-01-12 21:50:13]

sadrul@chromium.org changed reviewers:
+ sky@chromium.org

[sadrul, 2016-01-12 21:50:13]

No new tests: I am thinking the existing tests and the DCHECK()s provide
sufficient test coverage?

[sky, 2016-01-12 22:36:08]

LGTM with a better comment

https://codereview.chromium.org/1580193002/diff/1/components/mus/ws/window_tr...
File components/mus/ws/window_tree_impl.cc (right):

https://codereview.chromium.org/1580193002/diff/1/components/mus/ws/window_tr...
components/mus/ws/window_tree_impl.cc:418: // Do not notify ordering changes of
the root windows.
Document why. If window or relative_window is a root then the client can't see
above the window and so doesn't need to know about the change.

[sadrul, 2016-01-13 02:26:05]

https://codereview.chromium.org/1580193002/diff/1/components/mus/ws/window_tr...
File components/mus/ws/window_tree_impl.cc (right):

https://codereview.chromium.org/1580193002/diff/1/components/mus/ws/window_tr...
components/mus/ws/window_tree_impl.cc:418: // Do not notify ordering changes of
the root windows.
On 2016/01/12 22:36:08, sky wrote:
> Document why. If window or relative_window is a root then the client can't see
> above the window and so doesn't need to know about the change.

Done.

[sadrul, 2016-01-13 02:27:56]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-01-13 02:27:57]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1580193002/#ps20001
(title: ".")

[commit-bot: I haz the power, 2016-01-13 02:28:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1580193002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1580193002/20001

[commit-bot: I haz the power, 2016-01-13 03:47:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-13 03:48:02]

Description was changed from

==========
mus: Do not notify clients when the roots are reordered.

BUG=none
==========

to

==========
mus: Do not notify clients when the roots are reordered.

BUG=none

Committed: https://crrev.com/04a9cf00d60912dde3e6fe4caa66b205d14acb68
Cr-Commit-Position: refs/heads/master@{#369098}
==========

[commit-bot: I haz the power, 2016-01-13 03:48:02]

Patchset 2 (id:??) landed as
https://crrev.com/04a9cf00d60912dde3e6fe4caa66b205d14acb68
Cr-Commit-Position: refs/heads/master@{#369098}