Routes calls to WM through WS

Routes calls to WM through WS

By doing this we ensure the WM sees changes to the underlying windows
before any calls to the WM are received.

BUG=none
TEST=none
R=ben@chromium.org

Committed: https://crrev.com/40766911c1bccdb7ee615616fc9377abe121b308
Cr-Commit-Position: refs/heads/master@{#356429}

[sky, 2015-10-27 19:26:16]

I need to merge with your patch, but this is enough to give you a feel for how
things will be structured:

. WS knows about WM by way of WindowTreeHostFactory::CreateWindowTreeHost taking
a WindowManager.
. Clients still connect to the WM directly to open a new window. Generally at
the point clients want to open a window they don't necessarily have a connection
to the WS yet, so it seems as though window creation should still occur as we
had. By connecting to the WM first, we also continue to have the existing flow
of the WM starting first and then contacting the WS.
. I put WM functions directly on Window. This is the part I'm not too happy
about.
. NativeWidgetMus doesn't use the WM anymore, it calls through to Window
functions.

[sky, 2015-10-27 20:51:10]

Merged with your changes, ready to review.

[Ben Goodger (Google), 2015-10-27 22:25:05]

lgtm

https://codereview.chromium.org/1423133003/diff/40001/components/mus/example/...
File components/mus/example/wm/window_manager_application.cc (right):

https://codereview.chromium.org/1423133003/diff/40001/components/mus/example/...
components/mus/example/wm/window_manager_application.cc:54: for (auto request :
requests_) {
nit: no braces

[sky, 2015-10-27 22:30:27]

https://codereview.chromium.org/1423133003/diff/40001/components/mus/example/...
File components/mus/example/wm/window_manager_application.cc (right):

https://codereview.chromium.org/1423133003/diff/40001/components/mus/example/...
components/mus/example/wm/window_manager_application.cc:54: for (auto request :
requests_) {
On 2015/10/27 22:25:05, Ben Goodger (Google) wrote:
> nit: no braces

Done.

[sky, 2015-10-27 22:31:29]

The CQ bit was checked by sky@chromium.org

[sky, 2015-10-27 22:31:30]

The patchset sent to the CQ was uploaded after l-g-t-m from ben@chromium.org
Link to the patchset: https://codereview.chromium.org/1423133003/#ps60001
(title: "no braces")

[commit-bot: I haz the power, 2015-10-27 22:33:35]

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

[commit-bot: I haz the power, 2015-10-27 23:27:32]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-27 23:28:18]

Patchset 4 (id:??) landed as
https://crrev.com/40766911c1bccdb7ee615616fc9377abe121b308
Cr-Commit-Position: refs/heads/master@{#356429}