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

Unified Diff: media/filters/audio_file_reader.cc

Issue 13991013: Linux: make it possible to compile against libav when use_system_ffmpeg==1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: #ifdefs Created 7 years, 8 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 | « no previous file | media/filters/ffmpeg_audio_decoder.cc » ('j') | media/filters/ffmpeg_audio_decoder.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/audio_file_reader.cc
diff --git a/media/filters/audio_file_reader.cc b/media/filters/audio_file_reader.cc
index 32627b04b10ad558bab4b4facc386da0a3c4552d..fc39bfc1b2f38b1b77620f6e027e828c4f667eb7 100644
--- a/media/filters/audio_file_reader.cc
+++ b/media/filters/audio_file_reader.cc
@@ -180,12 +180,22 @@ int AudioFileReader::Read(AudioBus* audio_bus) {
}
if (av_frame->sample_rate != sample_rate_ ||
+#ifdef CHROMIUM_NO_AVFRAME_CHANNELS
DaleCurtis 2013/04/17 22:14:23 Instead of doing this, just do one #ifdef above th
Paweł Hajdan Jr. 2013/04/18 17:27:10 Done.
+ av_get_channel_layout_nb_channels(
+ av_frame->channel_layout) != channels_ ||
+#else
av_frame->channels != channels_ ||
+#endif
av_frame->format != av_sample_format_) {
DLOG(ERROR) << "Unsupported midstream configuration change!"
<< " Sample Rate: " << av_frame->sample_rate << " vs "
<< sample_rate_
+#ifdef CHROMIUM_NO_AVFRAME_CHANNELS
+ << ", Channels: " << av_get_channel_layout_nb_channels(
+ av_frame->channel_layout) << " vs "
+#else
<< ", Channels: " << av_frame->channels << " vs "
+#endif
<< channels_
<< ", Sample Format: " << av_frame->format << " vs "
<< av_sample_format_;
« no previous file with comments | « no previous file | media/filters/ffmpeg_audio_decoder.cc » ('j') | media/filters/ffmpeg_audio_decoder.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698