Register MediaPlayerRenderer service

media/mojo/clients/mojo_renderer_factory.cc

Index: media/mojo/clients/mojo_renderer_factory.cc
diff --git a/media/mojo/clients/mojo_renderer_factory.cc b/media/mojo/clients/mojo_renderer_factory.cc
index 750b5140d69f0cc30b30ee8c678d93a6ccacdf38..69aaea296b9e6fd4936c68920b7f89eab30915de 100644
--- a/media/mojo/clients/mojo_renderer_factory.cc
+++ b/media/mojo/clients/mojo_renderer_factory.cc
@@ -17,7 +17,6 @@ MojoRendererFactory::MojoRendererFactory(
     shell::mojom::InterfaceProvider* interface_provider)
     : get_gpu_factories_cb_(get_gpu_factories_cb),
       interface_provider_(interface_provider) {
-  DCHECK(!get_gpu_factories_cb_.is_null());
   DCHECK(interface_provider_);
 }
 
@@ -29,8 +28,12 @@ std::unique_ptr<Renderer> MojoRendererFactory::CreateRenderer(
     AudioRendererSink* /* audio_renderer_sink */,
     VideoRendererSink* video_renderer_sink,
     const RequestSurfaceCB& /* request_surface_cb */) {
-  std::unique_ptr<VideoOverlayFactory> overlay_factory(
-      new VideoOverlayFactory(get_gpu_factories_cb_.Run()));
+  std::unique_ptr<VideoOverlayFactory> overlay_factory;
+
+  if (!get_gpu_factories_cb_.is_null()) {

[xhwang, 2016/10/12 20:14:15]

Please add a comment when this could be null.

[tguilbert, 2016/10/12 21:46:25]

Done.

+    overlay_factory =
+        base::MakeUnique<VideoOverlayFactory>(get_gpu_factories_cb_.Run());
+  }
 
   mojom::RendererPtr renderer_ptr;
   shell::GetInterface<mojom::Renderer>(interface_provider_, &renderer_ptr);