Moves move logic into WM instead of WS

Moves move logic into WM instead of WS

The WS still needs to know about client area, which feels a bit
unfortunate.

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

Committed: https://crrev.com/d4ca12f08e8ad8c98ad4051e88a183b368cc5938
Cr-Commit-Position: refs/heads/master@{#356586}

[Ben Goodger (Google), 2015-10-28 16:03:16]

lgtm

https://codereview.chromium.org/1414943003/diff/1/components/mus/example/wm/w...
File components/mus/example/wm/window_manager_impl.cc (right):

https://codereview.chromium.org/1414943003/diff/1/components/mus/example/wm/w...
components/mus/example/wm/window_manager_impl.cc:57:
child_window->AddObserver(this);
can be changed later (since it'll likely conflict with my CL) but I wonder if
this should be limited to the WindowLayout & related classes, rather than done
here for all windows. not all windows in all containers are movable.

[sky, 2015-10-28 16:17:56]

https://codereview.chromium.org/1414943003/diff/1/components/mus/example/wm/w...
File components/mus/example/wm/window_manager_impl.cc (right):

https://codereview.chromium.org/1414943003/diff/1/components/mus/example/wm/w...
components/mus/example/wm/window_manager_impl.cc:57:
child_window->AddObserver(this);
On 2015/10/28 16:03:16, Ben Goodger (Google) wrote:
> can be changed later (since it'll likely conflict with my CL) but I wonder if
> this should be limited to the WindowLayout & related classes, rather than done
> here for all windows. not all windows in all containers are movable.

I think we need a windowtype enum that indicates this.

[sky, 2015-10-28 16:18:02]

The CQ bit was checked by sky@chromium.org

[sky, 2015-10-28 16:18:02]

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/1414943003/#ps20001
(title: "fix build")

[commit-bot: I haz the power, 2015-10-28 16:18:15]

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

[commit-bot: I haz the power, 2015-10-28 16:45:54]

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

[commit-bot: I haz the power, 2015-10-28 16:45:54]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sky, 2015-10-28 17:01:37]

The CQ bit was checked by sky@chromium.org

[sky, 2015-10-28 17:01:38]

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/1414943003/#ps40001
(title: "Fix compile")

[commit-bot: I haz the power, 2015-10-28 17:03:12]

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

[commit-bot: I haz the power, 2015-10-28 18:22:36]

Committed patchset #3 (id:40001)

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

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