Pass Connector* into MojoMediaClient::Initialize().

media/mojo/interfaces/media_service.mojom

Index: media/mojo/interfaces/media_service.mojom
diff --git a/media/mojo/interfaces/media_service.mojom b/media/mojo/interfaces/media_service.mojom
index 3acfa3aef67c7dc1e375e76c3079a7250e642fec..952a957be98849c06bb9e0e4ca77165d09605a17 100644
--- a/media/mojo/interfaces/media_service.mojom
+++ b/media/mojo/interfaces/media_service.mojom
@@ -8,6 +8,10 @@ import "media/mojo/interfaces/interface_factory.mojom";
 import "services/service_manager/public/interfaces/interface_provider.mojom";
 
 interface MediaService {
-  CreateInterfaceFactory(InterfaceFactory& factory,
-                       service_manager.mojom.InterfaceProvider frame_interfaces);
+  // Request an InterfaceFactory. |frame_interfaces| can optionally be used to

[xhwang, 2016/11/30 18:49:31]

nit: Request_s_

[slan, 2016/12/01 00:47:22]

Done.

+  // provide interfaces hosted by the caller to the remote InterfaceFactory
+  // implementation.
+  CreateInterfaceFactory(
+      InterfaceFactory& factory,
+      service_manager.mojom.InterfaceProvider? frame_interfaces);

[alokp, 2016/11/30 17:36:20]

nit: I do not quite like the name frame_interfaces, which kind of suggests that
this interface is somehow tied to render frames. We should make it more generic,
something like, remote_interfaces, maybe?

[slan, 2016/11/30 17:40:39]

client_interfaces? (As Xiaohan mentioned somewhere else, this should be
deprecated in favor of a delegate interface)

[xhwang, 2016/11/30 18:49:31]

Naming is hard :)

The context is here:
https://bugs.chromium.org/p/chromium/issues/detail?id=660573

This interface is provided by whoever calling CreateInterfaceFactory, which may
or may not be the final client of the InterfaceFactory. So client_interfaces is
a bit ambiguous. I think "remote_interfaces" would be better. Today the only
place providing this is in a "frame", hence the current name.

Maybe we should just introduce a new term, e.g. "host", which is the caller of
CreateInterfaceFactory(), then this would be "host_interfaces".

But given the fact that we want to replace this altogether, it's totally up to
you whether you want to do the rename or not in this CL.

[slan, 2016/12/01 00:47:22]

I like host_interfaces. Done.

 };