Add audio_device_id in mojo media ServiceFactory CreateRenderer api.

chromecast/browser/media/cast_renderer.cc

 // Copyright 2016 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 "chromecast/browser/media/cast_renderer.h"
 
 #include "base/bind.h"
 #include "base/single_thread_task_runner.h"
 #include "chromecast/base/task_runner_impl.h"
 #include "chromecast/media/cdm/cast_cdm_context.h"
@@ skipping 10 matching lines @@
 #include "media/base/media_log.h"
 #include "media/base/renderer_client.h"
 
 namespace chromecast {
 namespace media {
 
 namespace {
 // Maximum difference between audio frame PTS and video frame PTS
 // for frames read from the DemuxerStream.
 const base::TimeDelta kMaxDeltaFetcher(base::TimeDelta::FromMilliseconds(2000));
+
+int GetStreamType(const std::string& audio_device_id) {

[alokp, 2016/07/12 05:01:30]

This is weird. Where are these stream-type integers defined?

[tianyuwang1, 2016/07/12 15:28:09]

They are not defined in cast_shell, because we didn't want to leak the enums.  
That's why we were using an integer.  They are defined in the caller right now
(Assistant code).   We can switch over to use audio_device_id directly in
MediaBackendPipelineManager.  But I would prefer to do it in a separate cl,
since we need to change multiple places in order to switch from stream_type to
audio_device_id.

I've changed the code so that we can do the mapping between audio_device_id to
stream_type in CastContentBrowserInternal for now.

+  if (audio_device_id ==
+      media::AudioDeviceDescription::kCommunicationsDeviceId) {
+    return 1;
+  } else {
+    return 0;
+  }
+}
+
 }  // namespace
 
 CastRenderer::CastRenderer(
     const CreateMediaPipelineBackendCB& create_backend_cb,
-    const scoped_refptr<base::SingleThreadTaskRunner>& task_runner)
+    const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
+    const std::string& audio_device_id)
     : create_backend_cb_(create_backend_cb),
       task_runner_(task_runner),
+      audio_device_id_(audio_device_id),
       client_(nullptr),
       cast_cdm_context_(nullptr),
       media_task_runner_factory_(
           new BalancedMediaTaskRunnerFactory(kMaxDeltaFetcher)),
       weak_factory_(this) {
   CMALOG(kLogControl) << __FUNCTION__ << ": " << this;
 }
 
 CastRenderer::~CastRenderer() {
   CMALOG(kLogControl) << __FUNCTION__ << ": " << this;
@@ skipping 10 matching lines @@
   // Create pipeline backend.
   backend_task_runner_.reset(new TaskRunnerImpl());
   // TODO(erickung): crbug.com/443956. Need to provide right LoadType.
   LoadType load_type = kLoadTypeMediaSource;
   MediaPipelineDeviceParams::MediaSyncType sync_type =
       (load_type == kLoadTypeMediaStream)
           ? MediaPipelineDeviceParams::kModeIgnorePts
           : MediaPipelineDeviceParams::kModeSyncPts;
   MediaPipelineDeviceParams params(sync_type, backend_task_runner_.get());
   std::unique_ptr<MediaPipelineBackend> backend =
-      create_backend_cb_.Run(params);
+      create_backend_cb_.Run(params, GetStreamType(audio_device_id_));
 
   // Create pipeline.
   MediaPipelineClient pipeline_client;
   pipeline_client.error_cb =
       base::Bind(&CastRenderer::OnError, weak_factory_.GetWeakPtr());
   pipeline_client.buffering_state_cb = base::Bind(
       &CastRenderer::OnBufferingStateChange, weak_factory_.GetWeakPtr());
   pipeline_.reset(new MediaPipelineImpl());
   pipeline_->SetClient(pipeline_client);
   pipeline_->Initialize(load_type, std::move(backend));
@@ skipping 156 matching lines @@
   client_->OnVideoNaturalSizeChange(size);
 }
 
 void CastRenderer::OnVideoOpacityChange(bool opaque) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   client_->OnVideoOpacityChange(opaque);
 }
 
 }  // namespace media
 }  // namespace chromecast