aura-mus: Hook up PerformWindowMove().

aura-mus: Hook up PerformWindowMove().

We can now drag tabs out and dock them on the side again.

BUG=663527
Review-Url: https://codereview.chromium.org/2657003004
Cr-Commit-Position: refs/heads/master@{#446757}
Committed: https://chromium.googlesource.com/chromium/src/+/67981c6373076a89183211bf23b741101503672f

[Elliot Glaysher, 2017-01-26 23:31:21]

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

[commit-bot: I haz the power, 2017-01-26 23:32:42]

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-27 02:27:48]

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

[commit-bot: I haz the power, 2017-01-27 02:27:49]

Dry run: This issue passed the CQ dry run.

[Elliot Glaysher, 2017-01-27 18:45:30]

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

[commit-bot: I haz the power, 2017-01-27 18:45:55]

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

[Elliot Glaysher, 2017-01-27 18:47:13]

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

[Elliot Glaysher, 2017-01-27 18:47:13]

The ws side tests are still there. But the entire client side RunMoveLoop() code
doesn't appear to have any tests for it on any platform.

[sky, 2017-01-27 19:00:31]

https://codereview.chromium.org/2657003004/diff/20001/ui/aura/mus/window_tree...
File ui/aura/mus/window_tree_client.cc (right):

https://codereview.chromium.org/2657003004/diff/20001/ui/aura/mus/window_tree...
ui/aura/mus/window_tree_client.cc:1718:
PerformWindowMove(window_tree_host->window(),
Is there any reason to keep the PerformWindowMove/CAncelWindowMove functions
instead of moving the implementation here? I likely kept them when this file was
forked from services/ui/public/cpp/window_Tree_client and I suspect no one calls
them directly.

[Elliot Glaysher, 2017-01-27 19:20:41]

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

[commit-bot: I haz the power, 2017-01-27 19:21:28]

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-27 20:19:37]

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

[commit-bot: I haz the power, 2017-01-27 20:19:38]

Dry run: This issue passed the CQ dry run.

[Elliot Glaysher, 2017-01-27 20:22:16]

https://codereview.chromium.org/2657003004/diff/20001/ui/aura/mus/window_tree...
File ui/aura/mus/window_tree_client.cc (right):

https://codereview.chromium.org/2657003004/diff/20001/ui/aura/mus/window_tree...
ui/aura/mus/window_tree_client.cc:1718:
PerformWindowMove(window_tree_host->window(),
On 2017/01/27 19:00:31, sky wrote:
> Is there any reason to keep the PerformWindowMove/CAncelWindowMove functions
> instead of moving the implementation here? I likely kept them when this file
was
> forked from services/ui/public/cpp/window_Tree_client and I suspect no one
calls
> them directly.

There are not. Moved implementations.

[sky, 2017-01-27 20:26:41]

LGTM

[Elliot Glaysher, 2017-01-27 20:38:35]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2017-01-27 20:39:08]

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-27 20:44:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485549515314340,
"parent_rev": "521884b60a3ecb3b6ca5081b1e36244c2877e279", "commit_rev":
"67981c6373076a89183211bf23b741101503672f"}

[commit-bot: I haz the power, 2017-01-27 20:44:55]

Description was changed from

==========
aura-mus: Hook up PerformWindowMove().

We can now drag tabs out and dock them on the side again.

BUG=663527
==========

to

==========
aura-mus: Hook up PerformWindowMove().

We can now drag tabs out and dock them on the side again.

BUG=663527

Review-Url: https://codereview.chromium.org/2657003004
Cr-Commit-Position: refs/heads/master@{#446757}
Committed:
https://chromium.googlesource.com/chromium/src/+/67981c6373076a89183211bf23b7...
==========

[commit-bot: I haz the power, 2017-01-27 20:44:56]

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