mus: Make --use-mus-in-renderer work again.

mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none

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

[sadrul, 2016-06-02 04:57:26]

sadrul@chromium.org changed reviewers:
+ ben@chromium.org

[Ben Goodger (Google), 2016-06-02 17:48:56]

https://codereview.chromium.org/2037533002/diff/1/content/public/app/mojo/con...
File content/public/app/mojo/content_renderer_manifest.json (right):

https://codereview.chromium.org/2037533002/diff/1/content/public/app/mojo/con...
content/public/app/mojo/content_renderer_manifest.json:7: "app": [
"content::mojom::RenderWidgetWindowTreeClientFactory" ]
I think you should explicitly require this interface in the content_browser
manifest, rather than introducing it into the app class, which I'm trying to
keep clean of content-isms

[sadrul, 2016-06-03 07:44:11]

Description was changed from

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow chrome (or other "apps") to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none
==========

to

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none
==========

[sadrul, 2016-06-03 07:44:18]

https://codereview.chromium.org/2037533002/diff/1/content/public/app/mojo/con...
File content/public/app/mojo/content_renderer_manifest.json (right):

https://codereview.chromium.org/2037533002/diff/1/content/public/app/mojo/con...
content/public/app/mojo/content_renderer_manifest.json:7: "app": [
"content::mojom::RenderWidgetWindowTreeClientFactory" ]
On 2016/06/02 17:48:56, Ben Goodger (Google) wrote:
> I think you should explicitly require this interface in the content_browser
> manifest, rather than introducing it into the app class, which I'm trying to
> keep clean of content-isms

Done.

[Ben Goodger (Google), 2016-06-03 17:43:44]

lgtm

[sadrul, 2016-06-03 18:15:40]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-06-03 18:16:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037533002/20001

[commit-bot: I haz the power, 2016-06-03 18:21:08]

Description was changed from

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none
==========

to

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none
==========

[commit-bot: I haz the power, 2016-06-03 18:21:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-03 18:22:14]

Description was changed from

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none
==========

to

==========
mus: Make --use-mus-in-renderer work again.

. It is necessary to send the --use-mus-in-renderer flag to the renderer.
. Allow content_browser to connect to the renderer and get a handle to
  a RenderWidgetWindowTreeClientFactory.

BUG=none

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

[commit-bot: I haz the power, 2016-06-03 18:22:15]

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