Converts services/navigation to use aura-mus

Converts services/navigation to use aura-mus

This code won't actually work. There are a couple of things that need
to be fixed for it to work:

. My understanding from looking at the code is that it will create
  multiple WindowTreeClients in the same thread. That can work, but when
  you do that you have to explicitly pass a WindowPortMus to Windows
  when you create them. That's hard in this case because
  DesktopNativeWidgetAura is creating the windows for you. The fix
  isn't hard, probably set the WindowTreeClient in
  DesktopNativeWidgetAura so that it can create WindowPort correctly.

. aura::WindowTreeClientDelegate has a couple of functions that can't
  be implemented in ViewImpl. Specifically getting the capture client
  and a PropertyConverter. The capture client generally comes from
  WMState and the PropertyConverter can be a singleton. It shouldn't
  be that hard to wire these up.

. aura::Env needs to be created with Mode::MUS.

As my understanding is I can't really run this code yet I didn't
pursue the fixes. If there is a way to run this code I can try and
make these fixes, otherwise I would prefer to postpone.

BUG=674757
TEST=none
R=ben@chromium.org

Committed: https://crrev.com/5b18f736c212371ac7a8bbe985b44fbf018f1d06
Cr-Commit-Position: refs/heads/master@{#439036}

[sky, 2016-12-16 00:43:05]

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

[commit-bot: I haz the power, 2016-12-16 00:44:13]

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

[Ben Goodger (Google), 2016-12-16 00:46:29]

lgtm

[sky, 2016-12-16 00:47:05]

The CQ bit was unchecked by sky@chromium.org

[sky, 2016-12-16 00:47:09]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-12-16 00:47:35]

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, 2016-12-16 05:34:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481849229813560,
"parent_rev": "216d4b741ed1c0e039a03577fe5c3d8ad5137e50", "commit_rev":
"e1f236d8100837df8ee3c4cba586ebe03139f04e"}

[commit-bot: I haz the power, 2016-12-16 05:34:43]

Description was changed from

==========
Converts services/navigation to use aura-mus

This code won't actually work. There are a couple of things that need
to be fixed for it to work:

. My understanding from looking at the code is that it will create
  multiple WindowTreeClients in the same thread. That can work, but when
  you do that you have to explicitly pass a WindowPortMus to Windows
  when you create them. That's hard in this case because
  DesktopNativeWidgetAura is creating the windows for you. The fix
  isn't hard, probably set the WindowTreeClient in
  DesktopNativeWidgetAura so that it can create WindowPort correctly.

. aura::WindowTreeClientDelegate has a couple of functions that can't
  be implemented in ViewImpl. Specifically getting the capture client
  and a PropertyConverter. The capture client generally comes from
  WMState and the PropertyConverter can be a singleton. It shouldn't
  be that hard to wire these up.

. aura::Env needs to be created with Mode::MUS.

As my understanding is I can't really run this code yet I didn't
pursue the fixes. If there is a way to run this code I can try and
make these fixes, otherwise I would prefer to postpone.

BUG=674757
TEST=none
R=ben@chromium.org
==========

to

==========
Converts services/navigation to use aura-mus

This code won't actually work. There are a couple of things that need
to be fixed for it to work:

. My understanding from looking at the code is that it will create
  multiple WindowTreeClients in the same thread. That can work, but when
  you do that you have to explicitly pass a WindowPortMus to Windows
  when you create them. That's hard in this case because
  DesktopNativeWidgetAura is creating the windows for you. The fix
  isn't hard, probably set the WindowTreeClient in
  DesktopNativeWidgetAura so that it can create WindowPort correctly.

. aura::WindowTreeClientDelegate has a couple of functions that can't
  be implemented in ViewImpl. Specifically getting the capture client
  and a PropertyConverter. The capture client generally comes from
  WMState and the PropertyConverter can be a singleton. It shouldn't
  be that hard to wire these up.

. aura::Env needs to be created with Mode::MUS.

As my understanding is I can't really run this code yet I didn't
pursue the fixes. If there is a way to run this code I can try and
make these fixes, otherwise I would prefer to postpone.

BUG=674757
TEST=none
R=ben@chromium.org

Review-Url: https://codereview.chromium.org/2585593002
==========

[commit-bot: I haz the power, 2016-12-16 05:34:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-16 05:36:40]

Description was changed from

==========
Converts services/navigation to use aura-mus

This code won't actually work. There are a couple of things that need
to be fixed for it to work:

. My understanding from looking at the code is that it will create
  multiple WindowTreeClients in the same thread. That can work, but when
  you do that you have to explicitly pass a WindowPortMus to Windows
  when you create them. That's hard in this case because
  DesktopNativeWidgetAura is creating the windows for you. The fix
  isn't hard, probably set the WindowTreeClient in
  DesktopNativeWidgetAura so that it can create WindowPort correctly.

. aura::WindowTreeClientDelegate has a couple of functions that can't
  be implemented in ViewImpl. Specifically getting the capture client
  and a PropertyConverter. The capture client generally comes from
  WMState and the PropertyConverter can be a singleton. It shouldn't
  be that hard to wire these up.

. aura::Env needs to be created with Mode::MUS.

As my understanding is I can't really run this code yet I didn't
pursue the fixes. If there is a way to run this code I can try and
make these fixes, otherwise I would prefer to postpone.

BUG=674757
TEST=none
R=ben@chromium.org

Review-Url: https://codereview.chromium.org/2585593002
==========

to

==========
Converts services/navigation to use aura-mus

This code won't actually work. There are a couple of things that need
to be fixed for it to work:

. My understanding from looking at the code is that it will create
  multiple WindowTreeClients in the same thread. That can work, but when
  you do that you have to explicitly pass a WindowPortMus to Windows
  when you create them. That's hard in this case because
  DesktopNativeWidgetAura is creating the windows for you. The fix
  isn't hard, probably set the WindowTreeClient in
  DesktopNativeWidgetAura so that it can create WindowPort correctly.

. aura::WindowTreeClientDelegate has a couple of functions that can't
  be implemented in ViewImpl. Specifically getting the capture client
  and a PropertyConverter. The capture client generally comes from
  WMState and the PropertyConverter can be a singleton. It shouldn't
  be that hard to wire these up.

. aura::Env needs to be created with Mode::MUS.

As my understanding is I can't really run this code yet I didn't
pursue the fixes. If there is a way to run this code I can try and
make these fixes, otherwise I would prefer to postpone.

BUG=674757
TEST=none
R=ben@chromium.org

Committed: https://crrev.com/5b18f736c212371ac7a8bbe985b44fbf018f1d06
Cr-Commit-Position: refs/heads/master@{#439036}
==========

[commit-bot: I haz the power, 2016-12-16 05:36:41]

Patchset 2 (id:??) landed as
https://crrev.com/5b18f736c212371ac7a8bbe985b44fbf018f1d06
Cr-Commit-Position: refs/heads/master@{#439036}