Pass Connector* into MojoMediaClient::Initialize().

media/mojo/services/android_mojo_media_client.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/mojo/services/android_mojo_media_client.h"
 
 #include "base/memory/ptr_util.h"
 #include "media/base/android/android_cdm_factory.h"
 #include "media/base/audio_decoder.h"
 #include "media/base/cdm_factory.h"
@@ skipping 22 matching lines @@
 
 // MojoMediaClient overrides.
 
 std::unique_ptr<AudioDecoder> AndroidMojoMediaClient::CreateAudioDecoder(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner) {
   return base::MakeUnique<MediaCodecAudioDecoder>(task_runner);
 }
 
 std::unique_ptr<CdmFactory> AndroidMojoMediaClient::CreateCdmFactory(
     service_manager::mojom::InterfaceProvider* interface_provider) {
+  DCHECK(interface_provider) << "InterfaceProvider is not bound!";

[xhwang, 2016/11/30 18:52:50]

Given how flexible people can use mojo media, I imagine it's possible that
someone can use AndroidMojoMediaClient in a frame-less context, where this
DCEHCK will fire.

Can we actually do a if check here, and return nullptr if the interface_provider
is null?

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

To be honest, if I were a user of this interface, I would not expect that
behavior. But I will defer to your judgment. Done.

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