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

Unified Diff: media/ffmpeg/ffmpeg_common.h

Issue 286953005: Roll FFmpeg for M37. (Closed) Base URL: https://chromium.googlesource.com/chromium/src
Patch Set: Final DEPS. Created 6 years, 7 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: media/ffmpeg/ffmpeg_common.h
diff --git a/media/ffmpeg/ffmpeg_common.h b/media/ffmpeg/ffmpeg_common.h
index c697bee69f40772db371cd00a05a0d6e935969eb..ef1a7b6a7620cbabdf534be3797ea27842402c19 100644
--- a/media/ffmpeg/ffmpeg_common.h
+++ b/media/ffmpeg/ffmpeg_common.h
@@ -19,6 +19,14 @@
// Include FFmpeg header files.
extern "C" {
+// Disable deprecated features which result in spammy compile warnings. This
+// list of defines must mirror those in the 'defines' section of the ffmpeg.gyp
+// file or the headers below will generate different structures.
+#define FF_API_PIX_FMT_DESC 0
wolenetz 2014/05/20 22:37:10 I would prefer these be exported by ffmpeg.gyp and
DaleCurtis 2014/05/20 23:00:45 Hmm, I don't really like using direct_dependent_se
+#define FF_API_OLD_DECODE_AUDIO 0
+#define FF_API_DESTRUCT_PACKET 0
+#define FF_API_GET_BUFFER 0
+
// Temporarily disable possible loss of data warning.
// TODO(scherkus): fix and upstream the compiler warnings.
MSVC_PUSH_DISABLE_WARNING(4244);
@@ -61,7 +69,7 @@ inline void ScopedPtrAVFreeContext::operator()(void* x) const {
inline void ScopedPtrAVFreeFrame::operator()(void* x) const {
AVFrame* frame = static_cast<AVFrame*>(x);
- avcodec_free_frame(&frame);
+ av_frame_free(&frame);
}
// Converts an int64 timestamp in |time_base| units to a base::TimeDelta.

Powered by Google App Engine
This is Rietveld 408576698