Add audio_device_id in mojo media ServiceFactory CreateRenderer api.

chromecast/browser/media/cast_mojo_media_client.cc

Index: chromecast/browser/media/cast_mojo_media_client.cc
diff --git a/chromecast/browser/media/cast_mojo_media_client.cc b/chromecast/browser/media/cast_mojo_media_client.cc
index 2ae2bc9d60546a19b601712290dc1db81403e68d..f8116e5db7f5afb431df6e73cc1bdf94d9cfc572 100644
--- a/chromecast/browser/media/cast_mojo_media_client.cc
+++ b/chromecast/browser/media/cast_mojo_media_client.cc
@@ -9,33 +9,6 @@
 #include "media/base/media_log.h"
 #include "media/base/renderer_factory.h"
 
-namespace {
-class CastRendererFactory : public media::RendererFactory {
- public:
-  CastRendererFactory(
-      const chromecast::media::CreateMediaPipelineBackendCB& create_backend_cb,
-      const scoped_refptr<media::MediaLog>& media_log)
-      : create_backend_cb_(create_backend_cb), media_log_(media_log) {}
-  ~CastRendererFactory() final {}
-
-  std::unique_ptr<media::Renderer> CreateRenderer(
-      const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
-      const scoped_refptr<base::TaskRunner>& worker_task_runner,
-      media::AudioRendererSink* audio_renderer_sink,
-      media::VideoRendererSink* video_renderer_sink,
-      const media::RequestSurfaceCB& request_surface_cb) final {
-    DCHECK(!audio_renderer_sink && !video_renderer_sink);
-    return base::WrapUnique(new chromecast::media::CastRenderer(
-        create_backend_cb_, media_task_runner));
-  }
-
- private:
-  const chromecast::media::CreateMediaPipelineBackendCB create_backend_cb_;
-  scoped_refptr<media::MediaLog> media_log_;
-  DISALLOW_COPY_AND_ASSIGN(CastRendererFactory);
-};
-}  // namespace
-
 namespace chromecast {
 namespace media {
 
@@ -45,11 +18,13 @@ CastMojoMediaClient::CastMojoMediaClient(
 
 CastMojoMediaClient::~CastMojoMediaClient() {}
 
-std::unique_ptr<::media::RendererFactory>
-CastMojoMediaClient::CreateRendererFactory(
-    const scoped_refptr<::media::MediaLog>& media_log) {
-  return base::WrapUnique(
-      new CastRendererFactory(create_backend_cb_, media_log));
+std::unique_ptr<::media::Renderer> CastMojoMediaClient::CreateRenderer(
+    scoped_refptr<base::SingleThreadTaskRunner> media_task_runner,
+    scoped_refptr<::media::MediaLog> media_log,
+    const std::string& audio_device_id) {
+  // TODO(tianyuwang): pass audio_device_id to CastRenderer.
+  return base::WrapUnique(new chromecast::media::CastRenderer(

[dcheng, 2016/06/03 04:29:48]

return base::MakeUnique<chromecast::media::CastRenderer>(create_backend_cb_,
std::move(media_task_runner));

[tianyuwang1, 2016/06/03 18:31:49]

Done.

+      create_backend_cb_, media_task_runner));
 }
 
 }  // namespace media