Gets mash to use WmToplevelWindowEventHandler

Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests

Committed: https://crrev.com/532d3841d0bf88a5ee911b561b4c19e7666c2755
Cr-Commit-Position: refs/heads/master@{#395453}

[sky, 2016-05-23 20:29:13]

Description was changed from

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests
==========

to

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests
==========

[sky, 2016-05-23 20:29:14]

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

[James Cook, 2016-05-23 21:53:34]

LGTM

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/drag_window_res...
File mash/wm/drag_window_resizer.cc (right):

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/drag_window_res...
mash/wm/drag_window_resizer.cc:23: // http://crbug.com/613199.
Nice to see bugs filed for notimplementeds.

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/frame/move_even...
File mash/wm/frame/move_event_handler.cc (right):

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/frame/move_even...
mash/wm/frame/move_event_handler.cc:74: const int ht_location =
nit: maybe hit_location or hit_test_location or window_component?

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/shelf_layout_ma...
File mash/wm/shelf_layout_manager.cc (right):

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/shelf_layout_ma...
mash/wm/shelf_layout_manager.cc:38: // Phanton windows end up in this container,
ignore them.
nit: Phanton -> Phantom

[sky, 2016-05-23 22:02:43]

The CQ bit was checked by sky@chromium.org

[sky, 2016-05-23 22:02:43]

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

[commit-bot: I haz the power, 2016-05-23 22:03:17]

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

[sky, 2016-05-23 22:23:26]

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/shelf_layout_ma...
File mash/wm/shelf_layout_manager.cc (right):

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/shelf_layout_ma...
mash/wm/shelf_layout_manager.cc:38: // Phanton windows end up in this container,
ignore them.
On 2016/05/23 21:53:34, James Cook wrote:
> nit: Phanton -> Phantom

Done.

[sky, 2016-05-23 22:25:44]

I forgot to hit 'done' for this one too. Sorry about that.

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/frame/move_even...
File mash/wm/frame/move_event_handler.cc (right):

https://codereview.chromium.org/2002243002/diff/20001/mash/wm/frame/move_even...
mash/wm/frame/move_event_handler.cc:74: const int ht_location =
On 2016/05/23 21:53:34, James Cook wrote:
> nit: maybe hit_location or hit_test_location or window_component?

Done.

[commit-bot: I haz the power, 2016-05-23 23:14:14]

Description was changed from

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests
==========

to

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests
==========

[commit-bot: I haz the power, 2016-05-23 23:14:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-23 23:15:35]

Description was changed from

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests
==========

to

==========
Gets mash to use WmToplevelWindowEventHandler

And wires up a couple of ash LayoutManagers that the resize code uses.

BUG=603369
TEST=covered by tests

Committed: https://crrev.com/532d3841d0bf88a5ee911b561b4c19e7666c2755
Cr-Commit-Position: refs/heads/master@{#395453}
==========

[commit-bot: I haz the power, 2016-05-23 23:15:36]

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