Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4341)

Unified Diff: chromecast/media/service/cast_renderer.cc

Issue 2701613006: [Chromecast] Process streams with different post-processing. (Closed)
Patch Set: Fix nits Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromecast/media/service/cast_renderer.cc
diff --git a/chromecast/media/service/cast_renderer.cc b/chromecast/media/service/cast_renderer.cc
index 42964baf9931ca51a38fc5d8e885dda2567a16da..b04d7ddf745b76051b67d7d14e5d437b4cfdedb5 100644
--- a/chromecast/media/service/cast_renderer.cc
+++ b/chromecast/media/service/cast_renderer.cc
@@ -14,8 +14,10 @@
#include "chromecast/media/cma/base/demuxer_stream_adapter.h"
#include "chromecast/media/cma/pipeline/media_pipeline_impl.h"
#include "chromecast/media/cma/pipeline/video_pipeline_client.h"
+#include "chromecast/public/media/audio_device_ids.h"
#include "chromecast/public/media/media_pipeline_backend.h"
#include "chromecast/public/media/media_pipeline_device_params.h"
+#include "media/audio/audio_device_description.h"
#include "media/base/audio_decoder_config.h"
#include "media/base/demuxer_stream.h"
#include "media/base/media_log.h"
@@ -77,7 +79,15 @@ void CastRenderer::Initialize(::media::MediaResource* media_resource,
(load_type == kLoadTypeMediaStream)
? MediaPipelineDeviceParams::kModeIgnorePts
: MediaPipelineDeviceParams::kModeSyncPts;
- MediaPipelineDeviceParams params(sync_type, backend_task_runner_.get());
+ MediaPipelineDeviceParams params(
+ sync_type, MediaPipelineDeviceParams::kAudioStreamNormal,
+ audio_device_id_ /* name */, backend_task_runner_.get());
+
+ if (audio_device_id_ == kTtsAudioDeviceId ||
+ audio_device_id_ == kSystemAudioDeviceId) {
+ load_type = kLoadTypeLiveSource;
wzhong 2017/02/21 16:03:46 Why are those 2 streams live? I thought live is o
bshaya 2017/02/21 23:30:14 Live is for any stream that benefits from minimum
+ }
+
std::unique_ptr<MediaPipelineBackend> backend =
create_backend_cb_.Run(params, audio_device_id_);

Powered by Google App Engine
This is Rietveld 408576698