Removes/promotes functions from WmWindowMus to WmWindowAura

Removes/promotes functions from WmWindowMus to WmWindowAura

The aura implementation for many of these can be used as is.

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

Review-Url: https://codereview.chromium.org/2613793002
Cr-Commit-Position: refs/heads/master@{#441689}
Committed: https://chromium.googlesource.com/chromium/src/+/307e3589d858c808490ec46f5dad414a97b1a544

[sky, 2017-01-04 23:59:51]

The CQ bit was checked by sky@chromium.org to run a CQ dry run

[sky, 2017-01-05 00:00:01]

I'm hoping to remove all overrides in WmWindowMus, but this is a good starting
point.

[commit-bot: I haz the power, 2017-01-05 00:00:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 01:42:15]

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

[commit-bot: I haz the power, 2017-01-05 01:42:16]

Dry run: This issue passed the CQ dry run.

[James Cook, 2017-01-05 02:50:10]

LGTM but please move the test api files.

https://codereview.chromium.org/2613793002/diff/20001/ash/mus/BUILD.gn
File ash/mus/BUILD.gn (right):

https://codereview.chromium.org/2613793002/diff/20001/ash/mus/BUILD.gn#newcod...
ash/mus/BUILD.gn:216: "bridge/wm_window_aura_test_api.cc",
These files should either live in //ash/aura alongside wm_window_aura.h/cc or in
//ash/test.

[sky, 2017-01-05 16:17:01]

Can you take another look at the BUILD.gn changes to make sure you are ok with
it?

https://codereview.chromium.org/2613793002/diff/20001/ash/mus/BUILD.gn
File ash/mus/BUILD.gn (right):

https://codereview.chromium.org/2613793002/diff/20001/ash/mus/BUILD.gn#newcod...
ash/mus/BUILD.gn:216: "bridge/wm_window_aura_test_api.cc",
On 2017/01/05 02:50:09, James Cook wrote:
> These files should either live in //ash/aura alongside wm_window_aura.h/cc or
in
> //ash/test.
> 

I was a bit hesitant to do that as it means making this target depend upon
ash/test. But given the direction we're now going that seems ok.

[sky, 2017-01-05 16:17:02]

The CQ bit was checked by sky@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 16:17:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[James Cook, 2017-01-05 16:36:58]

LGTM

[sky, 2017-01-05 16:38:05]

The CQ bit was unchecked by sky@chromium.org

[sky, 2017-01-05 16:38:11]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-01-05 16:38:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 17:30:29]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483634291470260,
"parent_rev": "867f4553745f6106db762b4260397740760aaacf", "commit_rev":
"307e3589d858c808490ec46f5dad414a97b1a544"}

[commit-bot: I haz the power, 2017-01-05 17:30:59]

Description was changed from

==========
Removes/promotes functions from WmWindowMus to WmWindowAura

The aura implementation for many of these can be used as is.

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

to

==========
Removes/promotes functions from WmWindowMus to WmWindowAura

The aura implementation for many of these can be used as is.

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

Review-Url: https://codereview.chromium.org/2613793002
Cr-Commit-Position: refs/heads/master@{#441689}
Committed:
https://chromium.googlesource.com/chromium/src/+/307e3589d858c808490ec46f5dad...
==========

[commit-bot: I haz the power, 2017-01-05 17:31:00]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/307e3589d858c808490ec46f5dad...