Wires up WindowTreeClient::SetDisplayRoot()

Wires up WindowTreeClient::SetDisplayRoot()

Mush and mash will make use of this soon.

BUG=708279
TEST=covered by test
R=erg@chromium.org

Review-Url: https://codereview.chromium.org/2839743002
Cr-Commit-Position: refs/heads/master@{#466781}
Committed: https://chromium.googlesource.com/chromium/src/+/86bb8825f73b55690c94f3fca973462082715ae0

[sky, 2017-04-24 19:07:08]

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

[commit-bot: I haz the power, 2017-04-24 19:07:50]

Dry run: CQ is trying da patch.

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

[sky, 2017-04-24 19:11:06]

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

[commit-bot: I haz the power, 2017-04-24 19:11:45]

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-04-24 20:16:59]

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

[commit-bot: I haz the power, 2017-04-24 20:17:00]

Dry run: This issue passed the CQ dry run.

[Elliot Glaysher, 2017-04-24 21:23:17]

https://codereview.chromium.org/2839743002/diff/40001/ui/aura/mus/window_tree...
File ui/aura/mus/window_tree_host_mus_init_params.h (right):

https://codereview.chromium.org/2839743002/diff/40001/ui/aura/mus/window_tree...
ui/aura/mus/window_tree_host_mus_init_params.h:30: struct AURA_EXPORT
DisplayInitParams {
Make this a moveable type instead of something put in unique_ptr<>s?

[sky, 2017-04-24 21:27:15]

https://codereview.chromium.org/2839743002/diff/40001/ui/aura/mus/window_tree...
File ui/aura/mus/window_tree_host_mus_init_params.h (right):

https://codereview.chromium.org/2839743002/diff/40001/ui/aura/mus/window_tree...
ui/aura/mus/window_tree_host_mus_init_params.h:30: struct AURA_EXPORT
DisplayInitParams {
On 2017/04/24 21:23:17, Elliot Glaysher wrote:
> Make this a moveable type instead of something put in unique_ptr<>s?

I didn't do that for two reasons:
1. I wanted to make it clear in WindowTreeHostMusInitParams that
DisplayInitParams is optional.
2. For WindowTreeHostMus I didn't want the extra weight of a by value member
that is only needed during setup.

[Elliot Glaysher, 2017-04-24 21:35:28]

fair enough. lgtm.

[sky, 2017-04-24 21:37:00]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-04-24 21:37:47]

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-04-24 21:43:19]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493069820991060,
"parent_rev": "419509cbf0ac3a8a8595b3ce9e56daf75e5c2f0c", "commit_rev":
"86bb8825f73b55690c94f3fca973462082715ae0"}

[commit-bot: I haz the power, 2017-04-24 21:43:26]

Description was changed from

==========
Wires up WindowTreeClient::SetDisplayRoot()

Mush and mash will make use of this soon.

BUG=708279
TEST=covered by test
R=erg@chromium.org
==========

to

==========
Wires up WindowTreeClient::SetDisplayRoot()

Mush and mash will make use of this soon.

BUG=708279
TEST=covered by test
R=erg@chromium.org

Review-Url: https://codereview.chromium.org/2839743002
Cr-Commit-Position: refs/heads/master@{#466781}
Committed:
https://chromium.googlesource.com/chromium/src/+/86bb8825f73b55690c94f3fca973...
==========

[commit-bot: I haz the power, 2017-04-24 21:43:28]

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