Pass mojo::MediaRendererPtr in MojoRendererImpl's constructor.

mojo/services/html_viewer/webmediaplayer_factory.cc

Index: mojo/services/html_viewer/webmediaplayer_factory.cc
diff --git a/mojo/services/html_viewer/webmediaplayer_factory.cc b/mojo/services/html_viewer/webmediaplayer_factory.cc
index ba11f2da3dd682227aff60987cfc5c1dac5fd8fc..b40f04085d385c2bdf5819ce084c1d79eb0d669e 100644
--- a/mojo/services/html_viewer/webmediaplayer_factory.cc
+++ b/mojo/services/html_viewer/webmediaplayer_factory.cc
@@ -18,6 +18,7 @@
 #include "media/blink/webmediaplayer_impl.h"
 #include "media/blink/webmediaplayer_params.h"
 #include "media/filters/gpu_video_accelerator_factories.h"
+#include "media/mojo/interfaces/media_renderer.mojom.h"
 #include "media/mojo/services/mojo_renderer_impl.h"
 #include "mojo/public/interfaces/application/shell.mojom.h"
 
@@ -59,8 +60,13 @@ blink::WebMediaPlayer* WebMediaPlayerFactory::CreateMediaPlayer(
     ServiceProviderPtr media_renderer_service_provider;
     shell->ConnectToApplication("mojo:media",
                                 GetProxy(&media_renderer_service_provider));
-    renderer.reset(new media::MojoRendererImpl(
-        GetMediaThreadTaskRunner(), media_renderer_service_provider.get()));
+
+    mojo::MediaRendererPtr mojo_media_renderer;

[jamesr, 2014/12/05 23:12:31]

we're in namespace mojo here, so you can drop the "mojo::" from this line and
the next

[xhwang, 2014/12/05 23:31:00]

Done.

+    mojo::ConnectToService(media_renderer_service_provider.get(),
+                           &mojo_media_renderer);
+
+    renderer.reset(new media::MojoRendererImpl(GetMediaThreadTaskRunner(),
+                                               mojo_media_renderer.Pass()));
   }
 
   media::WebMediaPlayerParams params(