Adds WmLayoutManager and the aura wrapper AuraLayoutManagerAdapter

Adds WmLayoutManager and the aura wrapper AuraLayoutManagerAdapter

No tests as ash_unittests is going to end up testing this by virtue
of converting some of the ash LayoutManagers to be WmLayoutManagers
and installing on aura::Windows.

BUG=603369
TEST=none
R=jamescook@chromium.org

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

[sky, 2016-04-15 16:37:49]

This is relative to https://codereview.chromium.org/1890713002/ .

[sky, 2016-04-15 17:13:40]

Now that other patch landed this is no longer relative.

[James Cook, 2016-04-15 17:44:35]

LGTM with nits

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/aura/aura_layout...
File ash/wm/aura/aura_layout_manager_adapter.h (right):

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/aura/aura_layout...
ash/wm/aura/aura_layout_manager_adapter.h:19: // AuraLayoutManagerAdapter is an
aura::LayoutManager that calls to to
nit: "to to" -> "to" or "into"

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
File ash/wm/common/wm_layout_manager.h (right):

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:19: class ASH_EXPORT WmLayoutManager {
nit: Class comment. What does this do and why must it exist?

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:35: // Invoked when the |SetVisible()| is
invoked on the window |child|.
nit: "when the |SetVisible()|" -> "when |SetVisible()|" or similar.

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:37: // window->IsVisible() may still return
false. See description in
nit: Did you mean child->IsVisible()?

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:38: // Window::IsVisible() for details.
nit: Which "Window" do you mean here? aura::Window? WmWindow? If the latter, the
comment is on WmWindow::GetTargetVisibility().

[sky, 2016-04-15 19:26:36]

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/aura/aura_layout...
File ash/wm/aura/aura_layout_manager_adapter.h (right):

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/aura/aura_layout...
ash/wm/aura/aura_layout_manager_adapter.h:19: // AuraLayoutManagerAdapter is an
aura::LayoutManager that calls to to
On 2016/04/15 17:44:35, James Cook wrote:
> nit: "to to" -> "to" or "into"

Done.

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
File ash/wm/common/wm_layout_manager.h (right):

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:19: class ASH_EXPORT WmLayoutManager {
On 2016/04/15 17:44:35, James Cook wrote:
> nit: Class comment. What does this do and why must it exist?

Ya, sorry. This is mostly transitional and I figured it was obvious. But you're
right. Added some comments.

https://codereview.chromium.org/1894593002/diff/20001/ash/wm/common/wm_layout...
ash/wm/common/wm_layout_manager.h:35: // Invoked when the |SetVisible()| is
invoked on the window |child|.
On 2016/04/15 17:44:35, James Cook wrote:
> nit: "when the |SetVisible()|" -> "when |SetVisible()|" or similar.

Sorry, I copied this from aura. I've cleaned it all up.

[sky, 2016-04-15 19:26:51]

The CQ bit was checked by sky@chromium.org

[sky, 2016-04-15 19:26:52]

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/1894593002/#ps40001
(title: "merge and feedback")

[commit-bot: I haz the power, 2016-04-15 19:27:01]

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

[commit-bot: I haz the power, 2016-04-15 20:07:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-15 20:09:32]

Description was changed from

==========
Adds WmLayoutManager and the aura wrapper AuraLayoutManagerAdapter

No tests as ash_unittests is going to end up testing this by virtue
of converting some of the ash LayoutManagers to be WmLayoutManagers
and installing on aura::Windows.

BUG=603369
TEST=none
R=jamescook@chromium.org
==========

to

==========
Adds WmLayoutManager and the aura wrapper AuraLayoutManagerAdapter

No tests as ash_unittests is going to end up testing this by virtue
of converting some of the ash LayoutManagers to be WmLayoutManagers
and installing on aura::Windows.

BUG=603369
TEST=none
R=jamescook@chromium.org

Committed: https://crrev.com/d6078624276db684b84b6f1a3c9da536d05012c2
Cr-Commit-Position: refs/heads/master@{#387686}
==========

[commit-bot: I haz the power, 2016-04-15 20:09:33]

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