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

Unified Diff: media/filters/chunk_demuxer.cc

Issue 2773793002: Allow mp3 audio codec for mpeg2ts containers in MSE (Closed)
Patch Set: CR feedback Created 3 years, 9 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/base/audio_codecs.cc ('k') | media/filters/stream_parser_factory.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/chunk_demuxer.cc
diff --git a/media/filters/chunk_demuxer.cc b/media/filters/chunk_demuxer.cc
index e97de2c75d4c2f768bba0262c6112729420e2407..d55b2deef680dcc1592176e63b1476dd26f7954b 100644
--- a/media/filters/chunk_demuxer.cc
+++ b/media/filters/chunk_demuxer.cc
@@ -604,8 +604,11 @@ ChunkDemuxer::Status ChunkDemuxer::AddId(const std::string& id,
std::unique_ptr<media::StreamParser> stream_parser(
StreamParserFactory::Create(type, parsed_codec_ids, media_log_));
- if (!stream_parser)
+ if (!stream_parser) {
+ DVLOG(1) << __func__ << " failed: unsupported mime_type=" << type
+ << " codecs=" << codecs;
return ChunkDemuxer::kNotSupported;
+ }
std::unique_ptr<FrameProcessor> frame_processor(
new FrameProcessor(base::Bind(&ChunkDemuxer::IncreaseDurationIfNecessary,
« no previous file with comments | « media/base/audio_codecs.cc ('k') | media/filters/stream_parser_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698