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

Unified Diff: media/filters/decoder_stream_traits.cc

Issue 2712843002: media: Add DecoderConfigType in DecoderStreamTraits (Closed)
Patch Set: 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
« no previous file with comments | « media/filters/decoder_stream_traits.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/decoder_stream_traits.cc
diff --git a/media/filters/decoder_stream_traits.cc b/media/filters/decoder_stream_traits.cc
index 9577add9f044b70ce8fd17cd40fe79225db8aea8..a578f400afbfc34f4d2b1782a39a4403451f4a8e 100644
--- a/media/filters/decoder_stream_traits.cc
+++ b/media/filters/decoder_stream_traits.cc
@@ -35,6 +35,13 @@ scoped_refptr<DecoderStreamTraits<DemuxerStream::AUDIO>::OutputType>
return OutputType::CreateEOSBuffer();
}
+// static
+DecoderStreamTraits<DemuxerStream::AUDIO>::DecoderConfigType
+DecoderStreamTraits<DemuxerStream::AUDIO>::GetDecoderConfig(
+ DemuxerStream* stream) {
+ return stream->audio_decoder_config();
+}
+
DecoderStreamTraits<DemuxerStream::AUDIO>::DecoderStreamTraits(
const scoped_refptr<MediaLog>& media_log)
: media_log_(media_log) {}
@@ -49,13 +56,13 @@ void DecoderStreamTraits<DemuxerStream::AUDIO>::ReportStatistics(
void DecoderStreamTraits<DemuxerStream::AUDIO>::InitializeDecoder(
DecoderType* decoder,
- DemuxerStream* stream,
+ const DecoderConfigType& config,
+ bool /* low_delay */,
CdmContext* cdm_context,
const InitCB& init_cb,
const OutputCB& output_cb) {
- DCHECK(stream->audio_decoder_config().IsValidConfig());
- decoder->Initialize(stream->audio_decoder_config(), cdm_context, init_cb,
- output_cb);
+ DCHECK(config.IsValidConfig());
+ decoder->Initialize(config, cdm_context, init_cb, output_cb);
}
void DecoderStreamTraits<DemuxerStream::AUDIO>::OnStreamReset(
@@ -96,6 +103,13 @@ DecoderStreamTraits<DemuxerStream::VIDEO>::CreateEOSOutput() {
return OutputType::CreateEOSFrame();
}
+// static
+DecoderStreamTraits<DemuxerStream::VIDEO>::DecoderConfigType
+DecoderStreamTraits<DemuxerStream::VIDEO>::GetDecoderConfig(
+ DemuxerStream* stream) {
+ return stream->video_decoder_config();
+}
+
DecoderStreamTraits<DemuxerStream::VIDEO>::DecoderStreamTraits(
const scoped_refptr<MediaLog>& media_log)
// Randomly selected number of samples to keep.
@@ -121,14 +135,13 @@ void DecoderStreamTraits<DemuxerStream::VIDEO>::ReportStatistics(
void DecoderStreamTraits<DemuxerStream::VIDEO>::InitializeDecoder(
DecoderType* decoder,
- DemuxerStream* stream,
+ const DecoderConfigType& config,
+ bool low_delay,
CdmContext* cdm_context,
const InitCB& init_cb,
const OutputCB& output_cb) {
- DCHECK(stream->video_decoder_config().IsValidConfig());
- decoder->Initialize(stream->video_decoder_config(),
- stream->liveness() == DemuxerStream::LIVENESS_LIVE,
- cdm_context, init_cb, output_cb);
+ DCHECK(config.IsValidConfig());
+ decoder->Initialize(config, low_delay, cdm_context, init_cb, output_cb);
}
void DecoderStreamTraits<DemuxerStream::VIDEO>::OnStreamReset(
« no previous file with comments | « media/filters/decoder_stream_traits.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698