Pass Connector* into MojoMediaClient::Initialize().

media/mojo/services/android_mojo_media_client.cc

Index: media/mojo/services/android_mojo_media_client.cc
diff --git a/media/mojo/services/android_mojo_media_client.cc b/media/mojo/services/android_mojo_media_client.cc
index 318429904300bf8c6c3cf29aaf98cfaef1d4ad76..c7f0b5f0ffdedcc6525e71bb8cd5c973524299bd 100644
--- a/media/mojo/services/android_mojo_media_client.cc
+++ b/media/mojo/services/android_mojo_media_client.cc
@@ -18,9 +18,9 @@ namespace media {
 namespace {
 
 std::unique_ptr<ProvisionFetcher> CreateProvisionFetcher(
-    service_manager::mojom::InterfaceProvider* interface_provider) {
+    service_manager::mojom::InterfaceProvider* host_interfaces) {
   mojom::ProvisionFetcherPtr provision_fetcher_ptr;
-  service_manager::GetInterface(interface_provider, &provision_fetcher_ptr);
+  service_manager::GetInterface(host_interfaces, &provision_fetcher_ptr);
   return base::MakeUnique<MojoProvisionFetcher>(
       std::move(provision_fetcher_ptr));
 }
@@ -39,9 +39,13 @@ std::unique_ptr<AudioDecoder> AndroidMojoMediaClient::CreateAudioDecoder(
 }
 
 std::unique_ptr<CdmFactory> AndroidMojoMediaClient::CreateCdmFactory(
-    service_manager::mojom::InterfaceProvider* interface_provider) {
+    service_manager::mojom::InterfaceProvider* host_interfaces) {
+  // |host_interfaces| is needed for CDM. If it was not bound by the host,
+  // return nullptr.
+  if (!host_interfaces)
+    return nullptr;

[dcheng, 2016/12/02 08:52:51]

Just trying to understand the change here: does hitting this mean that there's a
programming error? Is there any reason why code would be calling this with no
host interfaces? Or does this client simply not know?

[slan, 2016/12/02 16:45:53]

|host_interfaces| is, at present, used to provide interfaces specific to the
render frame. This change provides flexibility to clients who do not need to
provide additional interfaces, or who know nothing about render frames, but
still want to consume the "media" service.

Xiaohan may be able to provide specifics on what the Android use cases for this
are (perhaps some media playback of non-protected content from another context).
On Chromecast, we have several applications that will benefit from this
flexibility.

This pattern deviates from how interfaces requests should be brokered between
services (crbug.com/660573). Making this parameter optional takes a step in the
right direction for correcting this behavior.

[xhwang, 2016/12/02 17:47:48]

dcheng: good point. I think when this happens, it's generally a programming
error, thus a DCHECK is typically the recommendation. However, as slan@
mentioned, with the flexibility of mojo, there are many different ways to config
the system to use the media service with this client, and I was afraid when the
system was misconfiged we could end up here. This is similar to the current
warning/failure in mojo connector where a client tries to connect to an
interface, but no interface implementation is registered.

With that, maybe this will make more sense?

if (!host_interfaces) {
  NOTREACHED() << "Host interface should be provided when using CDM with
AndroidMojoMediaClient";
  return nullptr;
}

[dcheng, 2016/12/02 18:00:30]

I feel like a NOTREACHED() will just end up mostly ignored?

Also, my uncertainty about what to do here comes from my confusion about the
process model in media: I poked around a bit, and I /think/ this is running in
the unprivileged process... but I'm not 100% certain either. But in that case,
it's the unprivileged process making a programming error, so it would be useful
to fail early instead?

   return base::MakeUnique<AndroidCdmFactory>(
-      base::Bind(&CreateProvisionFetcher, interface_provider));
+      base::Bind(&CreateProvisionFetcher, host_interfaces));
 }
 
 }  // namespace media