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

Unified Diff: media/filters/ffmpeg_audio_decoder.cc

Issue 2534193003: To M56: Roll src/third_party/ffmpeg/ 3c7a09882..cdf4accee (3188 commits). (Closed)
Patch Set: Created 4 years, 1 month 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/ffmpeg_aac_bitstream_converter_unittest.cc ('k') | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_audio_decoder.cc
diff --git a/media/filters/ffmpeg_audio_decoder.cc b/media/filters/ffmpeg_audio_decoder.cc
index 14308435c5eeb8e641bc8f28fc80169412ab9877..86f8d580a54a15daa580606c161728bbebb3bb71 100644
--- a/media/filters/ffmpeg_audio_decoder.cc
+++ b/media/filters/ffmpeg_audio_decoder.cc
@@ -392,6 +392,18 @@ bool FFmpegAudioDecoder::ConfigureDecoder() {
// Release existing decoder resources if necessary.
ReleaseFFmpegResources();
+ // Use OpusAudioDecoder for Opus for now, even if FFmpeg claims to support
+ // Opus decode. Failure to configure here should lead to fall-back to
+ // OpusAudioDecoder.
+ // TODO(wolenetz,dalecurtis): Remove OpusAudioDecoder and use
+ // FFmpegAudioDecoder instead for Opus.
+ if (config_.codec() == kCodecOpus) {
+ MEDIA_LOG(DEBUG, media_log_)
+ << "Opus decode via FFmpegAudioDecoder is disabled";
+ state_ = kUninitialized;
+ return false;
+ }
+
// Initialize AVCodecContext structure.
codec_context_.reset(avcodec_alloc_context3(NULL));
AudioDecoderConfigToAVCodecContext(config_, codec_context_.get());
« no previous file with comments | « media/filters/ffmpeg_aac_bitstream_converter_unittest.cc ('k') | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698