thakis@chromium.org [Wed, 15 Feb 2012 16:11:59 +0000 (16:11 +0000)]
Switch/enum warnings moved from -Wswitch-enum to -Wswitch in clang,
rename flag.
TBR=evan
Review URL: https://chromiumcodereview.appspot.com/9406013
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@122092
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Tue, 7 Feb 2012 01:15:51 +0000 (01:15 +0000)]
Avoid OOB reads in nal parsing.
BUG=112670
Review URL: https://chromiumcodereview.appspot.com/9340008
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@120671
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Mon, 6 Feb 2012 21:58:07 +0000 (21:58 +0000)]
Fix fairly harmless OOB read from MAV_0001.3G2.asan.4f.188
Review URL: https://chromiumcodereview.appspot.com/9309087
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@120624
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
rbultje@chromium.org [Thu, 2 Feb 2012 19:56:20 +0000 (19:56 +0000)]
Also slightly move around code not allocate a new frame if we won't
decode it. This prevents us from putting undecoded frames in frame
pointers, which (in mt decoding) other threads will use and wait on
as references, causing a deadlock (if we skipped decoding) or a crash
(if we didn't initialized next_framep[] at all).
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
BUG=112384
TEST=none
Review URL: http://codereview.chromium.org/9314018
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@120229
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Thu, 2 Feb 2012 03:37:20 +0000 (03:37 +0000)]
Don't compile in unnecessary code if the underlying codecs are disabled.
BUG=110849
Review URL: https://chromiumcodereview.appspot.com/9315033
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@120156
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scottmg@chromium.org [Mon, 30 Jan 2012 18:59:52 +0000 (18:59 +0000)]
Remove now-unnecessary cygwin target dependency.
Review URL: http://codereview.chromium.org/9298037
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@119698
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
rbultje@chromium.org [Mon, 23 Jan 2012 19:51:09 +0000 (19:51 +0000)]
Fix vorbis floor1[] vector integer overflow and subsequent overflows in
the inverse db lookup table, causing stack overreads.
BUG=108416
Review URL: http://codereview.chromium.org/9169023
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@118716
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
thakis@chromium.org [Tue, 17 Jan 2012 22:23:55 +0000 (22:23 +0000)]
Disable harmless warnings for 3rdparty code: ffmpeg.
BUG=102390
Review URL: https://chromiumcodereview.appspot.com/9225006
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@117977
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 29 Nov 2011 19:46:50 +0000 (19:46 +0000)]
Fix VP3 coefficient index overflow.
Patch is cherry picked from ffmpeg.git commit
eef5c35b4352ec49ca41f6198bee8a976b1f81e5.
BUG=105652
Review URL: http://codereview.chromium.org/8659028
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@111997
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 29 Nov 2011 00:22:25 +0000 (00:22 +0000)]
Fix whitespace around operators in ffmpeg.gyp and ffmpeg_generated.gypi.
GYP files are Python and should follow the Python style guide.
Review URL: http://codereview.chromium.org/8729011
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@111818
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Mon, 28 Nov 2011 23:08:28 +0000 (23:08 +0000)]
Use os_posix in gyp file, this patch has been tested on FreeBSD, Solaris, and NetBSD.
Patch by chromium@hybridsource.org:
http://codereview.chromium.org/8673004/
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@111804
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Wed, 9 Nov 2011 22:28:35 +0000 (22:28 +0000)]
Fix invalid reads detected by Valgrind when the frame size changes.
BUG=102789
Review URL: http://codereview.chromium.org/8497048
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@109319
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Fri, 28 Oct 2011 23:33:24 +0000 (23:33 +0000)]
An additional defense in the Vorbis codec.
BUG=101458
Review URL: http://codereview.chromium.org/8414025
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@107826
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Fri, 28 Oct 2011 00:13:50 +0000 (00:13 +0000)]
Fix vorbis decoder bug.
BUG=101458
Review URL: http://codereview.chromium.org/8413019
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@107662
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Thu, 27 Oct 2011 00:38:14 +0000 (00:38 +0000)]
Apply patches from 101172, 100465.
Update some patches so they apply with make_src_tree.sh.
Patch by rbultje@chromium.org:
http://codereview.chromium.org/8392015/
BUG=101172,100465
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@107489
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
mark@chromium.org [Mon, 24 Oct 2011 19:56:00 +0000 (19:56 +0000)]
add support for OpenBSD in ffmpeg
add openbsd config files
ffmpeg.gyp:
- use openbs config files
- disable -fno-signed-zeros because our gcc does not support it.
BUG=
TEST=
Patch by Robert Nagy <robert.nagy@gmail.com>
Review URL: http://codereview.chromium.org/8275011
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@106959
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Thu, 20 Oct 2011 23:11:04 +0000 (23:11 +0000)]
Avoid the possibility to read out-of-bounds of a static global array in Vorbis
decoding.
BUG=100543
Review URL: http://codereview.chromium.org/8365014
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@106621
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Thu, 20 Oct 2011 22:01:27 +0000 (22:01 +0000)]
Fix a bug where a pointer was cached to an array that might later move due to
a realloc()
BUG=100492
Review URL: http://codereview.chromium.org/8366004
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@106599
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 18 Oct 2011 00:34:20 +0000 (00:34 +0000)]
VP8: fix up handling of segmentation_maps in reference frames.
Associate segmentation_map[] with reference frame, rather than decoding instance. This fixes cases where the map would be free()'ed on e.g. a size change in one thread, whereas the other thread was still accessing it. Also, it fixes cases where threads overwrite data that is still being referenced by the previous thread, who thinks that it's part of the frame previously decoded by the next thread.
Patch by rbultje@chromium.org.
BUG=100464
TEST=run file in bug report
TBR=rbultje
Review URL: http://codereview.chromium.org/8341002
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105977
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Sat, 15 Oct 2011 03:50:34 +0000 (03:50 +0000)]
Parse matroska clusters incrementally as opposed to all at once.
While the patch is ugly, it prevents us from downloading the entire first cluster during the call to av_find_stream_info(). As a result, start up latency is greatly improved.
BUG=100439
TEST=start up time, seeking, everything works
Review URL: http://codereview.chromium.org/8296016
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105638
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Fri, 14 Oct 2011 18:04:54 +0000 (18:04 +0000)]
ffmpeg: fix out-of-bounds write in MKV demuxer if laces==1.
Patch by rbultje@chromium.org:
http://codereview.chromium.org/8295005/
BUG=99652
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105521
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Fri, 14 Oct 2011 04:06:30 +0000 (04:06 +0000)]
FFmpeg: add patch that fixes ffvp8 overreads/overwrites (round 2).
This patch fixes the failures reported by media_unittests.
Patch by rbultje@chromium.org:
http://codereview.chromium.org/8289001/
BUG=99652
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105457
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Thu, 13 Oct 2011 23:27:58 +0000 (23:27 +0000)]
FFmpeg: add patch that fixes ffvp8 overreads/overwrites.
Patch by rbultje@chromium.org:
http://codereview.chromium.org/8289001/
BUG=99652
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105409
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Wed, 12 Oct 2011 19:32:33 +0000 (19:32 +0000)]
Cleanup: no need to redeclare use_system_yasm.
BUG=100078
Review URL: http://codereview.chromium.org/8226030
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@105133
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 5 Oct 2011 06:41:40 +0000 (06:41 +0000)]
FFmpeg: Switch ffmpeg ARM compilation to use the exact same settings as are used in CrOS chroot, in particular we configure it for thumb which should make a difference on which FFmpeg #defines/assembly files are actually used.
This change used to break the gold linker (relocation overflow in R_ARM_THM_JUMP8), but this has been fixed and I verified it.
BUG=19698
Review URL: http://codereview.chromium.org/7825014
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@104071
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Tue, 4 Oct 2011 20:59:12 +0000 (20:59 +0000)]
Fix signature files so code looks for av_get_bits_per_sample_fmt() in the correct shared library.
Fixed --extra-ldflags on Windows to silence a warning message.
BUG=98714
TEST=None
Review URL: http://codereview.chromium.org/8142008
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@103983
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Fri, 30 Sep 2011 23:09:04 +0000 (23:09 +0000)]
Upstream change fixing seeking for WebM with CUES at the front.
BUG=98714
Review URL: http://codereview.chromium.org/8094010
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@103568
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Thu, 1 Sep 2011 03:35:33 +0000 (03:35 +0000)]
FFmpeg: Apply the VP8_arm_optimizations patch to the source tree.
BUG=crosbug.com/18265
TESTS=Verified it plays well on tegra2 with webm content (720p).
Verified I don't hit linking error of crosbug.com/19698
Review URL: http://codereview.chromium.org/7822010
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@99115
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 31 Aug 2011 19:28:18 +0000 (19:28 +0000)]
FFmpeg:
- Added mulaw audio codec for ChromeOS only.
- Add rbultje's patch 36_theora_flush.patch that should permit us to enable multi-threading.
- Add rbultje's patch 37_VP8_armv6_optimizations.patch.inactive without applying it to the source yet. It compiles and works well but I ran into problems on ARM with the gold linker. Delay wrestling with ARM to a later stage.
- Several patches were removed during previous commits but these changes did not make it to the source directory. This commit applies these previous changes to source and makes them go live. They should be noops.
- Currently copy_config.sh assumes that the user has created build.* directories for all 6 configurations. (mac, win, linux 32/64/arm/arm-neon). If only one configuration changes like here for ChromeOS, it is convenient to skip over all the others. Hence make copying conditional on existence and verbose.
BUG=18938
TESTS=Ran videos and wav files on Mario and tegra2.
Review URL: http://codereview.chromium.org/7738001
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@99019
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 31 Aug 2011 18:48:51 +0000 (18:48 +0000)]
Removing a TODO from ffmpeg.gyp
The vpx-less FFmpeg stubs no longer exist!
Review URL: http://codereview.chromium.org/7809023
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@99011
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
darin@chromium.org [Sun, 28 Aug 2011 04:09:31 +0000 (04:09 +0000)]
Create libffmpeg.so in the component build.
Review URL: http://codereview.chromium.org/7778004
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@98577
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 24 Aug 2011 17:08:56 +0000 (17:08 +0000)]
Fix ogg duration regression by reverting FFmpeg commit 5901cd6.
BUG=93620
TEST=build ffmpeg from source and run ./ffmpeg -i [file.ogg], duration should be correct
Review URL: http://codereview.chromium.org/7714039
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@98063
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Fri, 19 Aug 2011 05:34:46 +0000 (05:34 +0000)]
Remove libavutil/lfg.c from blacklist.
Looks like lfg.c is used on ffmpeg_branding=ChromeOS but strangely enough only the ChromeOS linker produced an unloadable FFmpeg binary while my desktop linker works fine. Strange.
BUG=92938
TEST=try media player / HTML5 audio+video on ChromeOS
TBR=ihf
Review URL: http://codereview.chromium.org/7684002
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@97428
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 17 Aug 2011 00:54:15 +0000 (00:54 +0000)]
Remove libvpx from FFmpeg in favour of ffvp8.
Clients in Chromium code can now link directly to libvpx without depending on FFmpeg / InitializeMediaLibrary().
BUG=50811
TEST=remoting_unittests, media_unittests, layout tests
Review URL: http://codereview.chromium.org/7659006
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@97071
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fischman@chromium.org [Tue, 16 Aug 2011 00:55:15 +0000 (00:55 +0000)]
Make clang build of ffmpegsumo silent.
BUG=none
TEST=building ffmpegsumo with ninja/clang/goma emits no warnings.
Review URL: http://codereview.chromium.org/7639021
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96868
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Mon, 15 Aug 2011 21:01:44 +0000 (21:01 +0000)]
Remove libavutil/fifo.c from generate_gyp.py blacklist.
The Chromium/Chrome OS libraries require fifo.c due to flac_parser.c.
BUG=92938
TEST=build ffmpeg libraries for chromeos=1 and run player_x11
Review URL: http://codereview.chromium.org/7645035
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96828
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Mon, 15 Aug 2011 19:42:06 +0000 (19:42 +0000)]
Add av_register_bitstream_filter() to signature list so it can be used by media_unittests.
Review URL: http://codereview.chromium.org/7646020
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96812
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
hclam@chromium.org [Thu, 11 Aug 2011 12:56:54 +0000 (12:56 +0000)]
FFmpeg to build with libvpx on arm.
Update build_ffmpeg.sh and generated gypi file to build with libvpx on arm.
TBR=scherkus
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96373
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Thu, 11 Aug 2011 00:42:23 +0000 (00:42 +0000)]
Un-revert the FFmpeg portion of r96164 now that DEPS has been successfully rolled.
TBR=hclam
Review URL: http://codereview.chromium.org/7462024
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96282
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 10 Aug 2011 23:35:36 +0000 (23:35 +0000)]
Revert the FFmpeg portion of r96164 so we can switch to ffmpeg_generated.gypi.
TBR=hclam
Review URL: http://codereview.chromium.org/7611010
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96257
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 10 Aug 2011 20:03:25 +0000 (20:03 +0000)]
Switching to auto-generated FFmpeg source list.
Maintaining a gyp listing of FFmpeg sources by hand is crazy. Instead, use the result of build_ffmpeg.sh on multiple platforms to construct pairwise disjoint sets of source files that apply to specific combinations of architecture and branding.
BUG=92246
TEST=generate_gyp_unittest.py
Review URL: http://codereview.chromium.org/7519025
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96215
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 10 Aug 2011 19:43:37 +0000 (19:43 +0000)]
Partial revert of r96164 to fix mac FFmpeg build.
-Wl,-read_only_relocs,suppress is still required to build at this time.
TBR=hclam
Review URL: http://codereview.chromium.org/7583045
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96212
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
hclam@chromium.org [Wed, 10 Aug 2011 10:18:08 +0000 (10:18 +0000)]
Fix libvpx.gyp to build libvpx on ARM
The gyp file had several errors that it don't actually build on arm with
neon enabled (there's no target_arch named arm-neon).
There were also some assembly files missing. We shouldn't generate them
hand but should convert them as a build step. This change removes those
converted assembly files and added rule to convert them in tree.
However the in-tree conversion requires a gyp changes to take effect:
http://codereview.chromium.org/7602015/
Finally this change allow FFmpeg to link with libvpx on ARM.
Review URL: http://codereview.chromium.org/7558025
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@96164
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Mon, 8 Aug 2011 19:06:04 +0000 (19:06 +0000)]
Add a missing include after codereview.chromium.org/7541081/
BUG=none
Review URL: http://codereview.chromium.org/7529029
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@95849
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Mon, 8 Aug 2011 17:54:32 +0000 (17:54 +0000)]
Get rid of use_system_vpx references in ffmpeg.gyp,
it's broken anyway. And broken is broken.
BUG=none
Review URL: http://codereview.chromium.org/7541081
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@95837
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
evan@chromium.org [Wed, 3 Aug 2011 19:08:20 +0000 (19:08 +0000)]
Delete dummy_nolink.cc, as it's no longer used.
dummy_nolink was to work around a gyp issue, but I've fixed the
gyp issue. This file is no longer mentioned in any build file.
Review URL: http://codereview.chromium.org/7562013
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@95281
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 27 Jul 2011 18:31:04 +0000 (18:31 +0000)]
Remove date comments from newly added FFmpeg files.
They add noise and don't serve a purpose that a source repository cannot provide.
TBR=ihf
Review URL: http://codereview.chromium.org/7465073
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@94325
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
evan@chromium.org [Tue, 26 Jul 2011 23:55:15 +0000 (23:55 +0000)]
Build ffmpegsumo as a loadable_module, not a shared_library
The former is the designated gyp target type for a module that is
dynamically loaded like ffmpegsumo is. By using the right target
type, we no longer need the ffmpegsumo_nolink hack.
WARNING: On Mac, the resulting filename is different than before.
To merge this change into Chrome, we must fix the paths used by
Chrome simultaneously when updating gyp.
BUG=74216
Review URL: http://codereview.chromium.org/7470028
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@94207
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 26 Jul 2011 22:55:47 +0000 (22:55 +0000)]
Clarify ffmpeg.gyp comments.
Review URL: http://codereview.chromium.org/7470032
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@94192
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 26 Jul 2011 20:40:47 +0000 (20:40 +0000)]
Fix arm chromium build. I have two reports that the Chromium configuration is missing an ffmpeg source file. This change moves the file to the proper location so it is available for both Chrome and Chromium.
BUG=88696, 88436, 88850
Review URL: http://codereview.chromium.org/7477019
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@94155
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 26 Jul 2011 01:47:51 +0000 (01:47 +0000)]
Explicitely remove -mthumb CFLAG as the Chromium bots add it breaking ffmpeg build (which does not happen in chroot).
crosbug.com/16026
BUG=88696
TESTS=unfortunately I am absolutely unable to test this without a Chromium ARM trybot.
Review URL: http://codereview.chromium.org/7508008
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@94018
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Fri, 22 Jul 2011 22:26:35 +0000 (22:26 +0000)]
Fix ffmpeg.gyp arm conditions syntax.
BUG=88696, 88436, 88850
TESTS=Verified chrome compiles on chroot for arm and also for x86.
Review URL: http://codereview.chromium.org/7461038
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@93715
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 20 Jul 2011 05:34:18 +0000 (05:34 +0000)]
Fix ffmpeg.gyp for ChromiumOS build.
BUG=88696
Review URL: http://codereview.chromium.org/7457015
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@93168
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 20 Jul 2011 00:57:40 +0000 (00:57 +0000)]
The try bots run the hooks with CHROMIUM_GYP_SYNTAX_CHECK=1 and this does not like
having two conitions keys at the same level. (Even though everything worked in
practice without that flag.)
BUG=88696
Review URL: http://codereview.chromium.org/7445008
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@93138
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 19 Jul 2011 04:34:16 +0000 (04:34 +0000)]
Adding new patched-ffmpeg source tree.
BUG=88696
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@92970
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 19 Jul 2011 04:07:04 +0000 (04:07 +0000)]
Remove patched-ffmpeg to make space for new sources.
BUG=88696
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@92964
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Tue, 19 Jul 2011 01:06:31 +0000 (01:06 +0000)]
ffmpeg: July sources.
1) New CrOS codecs to support everything Android.
2) ARM specific compile flags were missing and have been added.
3) Started patch cleaned up.
Compiles on all configurations
BUG=88696,88436,88850 and CrOS:17352,16026,17380
TESTS=ran videos on mario and tegra
Review URL: http://codereview.chromium.org/7388001
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@92935
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
evan@chromium.org [Thu, 14 Jul 2011 21:23:25 +0000 (21:23 +0000)]
Fix a warning about usleep by better matching upstream flags.
The buildbots have this warning:
warning: implicit declaration of function 'usleep'
usleep is in unistd.h, but *not* when you build with std=c99 like we do,
*unless* you add an extra #define like I have done, but *only* on an older glibc.
BUG=89186
Review URL: http://codereview.chromium.org/7368008
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@92602
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 29 Jun 2011 23:07:18 +0000 (23:07 +0000)]
Fix crash playing chained stream Theora Testsuite files.
BUG=87981
TESTS=ran chrome on repro case
Review URL: http://codereview.chromium.org/7273082
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@91042
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Sat, 25 Jun 2011 08:59:25 +0000 (08:59 +0000)]
Correctly handle use_system_vpx.
BUG=87266
Review URL: http://codereview.chromium.org/7242019
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@90498
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Wed, 22 Jun 2011 02:35:28 +0000 (02:35 +0000)]
Fix linking errors on mac build using new ffmpeg.
BUG=87030
TEST=built it on my mac box
Review URL: http://codereview.chromium.org/7190025
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@89965
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Mon, 20 Jun 2011 18:46:27 +0000 (18:46 +0000)]
Remove patched-ffmpeg-mt directory.
Add patched-ffmpeg directory.
Review URL: http://codereview.chromium.org/7202016
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@89704
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Mon, 20 Jun 2011 18:39:34 +0000 (18:39 +0000)]
Deleting deps/third_party/ffmpeg/source/patched-ffmpeg-mt/ to make way for updated FFmpeg source.
BUG=none
TEST=none
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@89701
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
ihf@chromium.org [Sat, 18 Jun 2011 01:38:21 +0000 (01:38 +0000)]
ffmpeg:
1) switched from ffmpeg-mt to ffmpeg
2) adjusted patches
3) updated scripts to generate, configure/make, copy
4) added Chrome/ChromeOS targets with AVI and MPEG4
5) generated new config.h files for every config
6) added config.asm files for ia32/x64 configs
7) updated ffmpeg.gyp file (Works on linux, CrOS ia32, CrOS arm and arm-neon.)
Note:
- ffmpeg.gyp not tested for Mac and Windows. Will use trybots to see if a file is missing.
- Directory source/patched-ffmpeg-mt not deleted nor source/patched-ffmpeg added yet, as this is a huge change and can be generated by running make_src_tree.sh. (But will be updated in a later checkin.)
Review URL: http://codereview.chromium.org/7189003
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@89578
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Wed, 8 Jun 2011 20:03:05 +0000 (20:03 +0000)]
Enforce get_bits buffer limits for mp3 too, turns out it's just as sloppy as
the other codecs.
BUG=85041
Review URL: http://codereview.chromium.org/7134022
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@88382
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
hclam@chromium.org [Mon, 6 Jun 2011 21:37:45 +0000 (21:37 +0000)]
Build libvpx in tree on Mac and Win
Build in tree is winning!
Review URL: http://codereview.chromium.org/7109019
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@88042
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Wed, 1 Jun 2011 17:49:09 +0000 (17:49 +0000)]
Removing av_rescale() from avutil-50.sigs as we're using av_q2d() for aspect ratio calculations.
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@87495
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 31 May 2011 18:15:04 +0000 (18:15 +0000)]
Add av_rescale to avutil-50.sigs.
This symbol is needed to fix 18941.
Patch by scottfr@chromium.org:
http://codereview.chromium.org/7053044/
BUG=18941
TEST=none
Review URL: http://codereview.chromium.org/7053044
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@87330
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Fri, 29 Apr 2011 18:15:01 +0000 (18:15 +0000)]
Defer parsing of CUES element until we actually need the index data.
These changes defer loading the seek index data until a seek is requested. This prevents unnecessary file seeks at the beginning of playback which results in improved startup latency.
BUG=80845
TEST=none
Review URL: http://codereview.chromium.org/6904095
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@83550
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
crogers@google.com [Wed, 27 Apr 2011 22:37:17 +0000 (22:37 +0000)]
Add support for FFmpeg's RDFT functions.
BUG=none
TEST=none
(tested locally so far on Mac OS X)
Review URL: http://codereview.chromium.org/6901016
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@83236
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
piman@google.com [Tue, 26 Apr 2011 01:18:45 +0000 (01:18 +0000)]
Add "-Wa,-mimplicit-it=always" option to ffmpeg.gyp which allows it to be compiled in thumb2 mode. This is not ideal (we should fix the assembly instead). Refer to the bug attached.
Following this CL, I will try to remove the -Wa,-mimplicit-it option from the high level build file.
TEST=Built chromium on chromiumos on seaboard. Checked that the option was being passed when compiling ffmpeg only.
BUG=80521
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@82962
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 19 Apr 2011 18:19:17 +0000 (18:19 +0000)]
Add vpx_img_alloc, vpx_img_free and vpx_codec_enc_config_set to avcodec-52.sigs.
These symbols are needed by another project.
Patch by holmer@google.com:
http://codereview.chromium.org/6871013/
BUG=none
TEST=none
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@82126
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 19 Apr 2011 18:18:03 +0000 (18:18 +0000)]
Set svn:executable on FFmpeg's configure script.
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@82125
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Mon, 18 Apr 2011 16:12:03 +0000 (16:12 +0000)]
Check for null point in mkv to avoid a crash. Move 27_swscaler_21504 patch to from_upstream
BUG=77313
TEST=play crash.mkv from crbug report. should play without crashing.
Review URL: http://codereview.chromium.org/6870028
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@81953
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Wed, 30 Mar 2011 17:35:47 +0000 (17:35 +0000)]
bump max width for swscaler from 5120 to 21504
BUG=none
TEST=ffmpeg -y -i tulip2.mp4 -s 12720x724 tulip12.yuv
Review URL: http://codereview.chromium.org/6731084
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@79856
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
acolwell@chromium.org [Wed, 30 Mar 2011 17:15:41 +0000 (17:15 +0000)]
Add av_index_search_timestamp() to symbols list.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6756039
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@79854
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Tue, 22 Mar 2011 16:24:53 +0000 (16:24 +0000)]
mkv duration fixBUG=76556TESTED=ffmpeg tool should not show estimated duration on mkv files
Review URL: http://codereview.chromium.org/6713060
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@78987
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Tue, 22 Mar 2011 16:09:28 +0000 (16:09 +0000)]
Reapplying patches these are the differences
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6715022
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@78986
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
hbono@chromium.org [Wed, 16 Mar 2011 08:16:48 +0000 (08:16 +0000)]
Support the 'use_system_yasm' variable (ffmpeg).
This change uses yasm installed to the system to build ffmpeg if 'use_system_yasm' is 1 as requested by 'fta@ubuntu.com'.
BUG=none
TEST=build CHromium
Review URL: http://codereview.chromium.org/6670027
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@78339
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Fri, 11 Mar 2011 07:47:13 +0000 (07:47 +0000)]
Linux: fixes to the system libvpx support.
The previous gyp snippets was wrong,
replaced it with new one properly tested.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6659038
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@77796
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Tue, 8 Mar 2011 18:54:17 +0000 (18:54 +0000)]
Add avcodec_alloc_context() to avcodec-52 function imports.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6637001
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@77311
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Wed, 2 Mar 2011 16:17:36 +0000 (16:17 +0000)]
Fix a build failure with bundled libvpx.
This is part of an effort to allow building with system libvpx.
I was only using trybots previously, so not all problems
got detected during the upstreaming, especially with
bundled libraries.
This is not ideal, but I just copied the include_dirs
snippet from elsewhere.
TBR=fischman, scherkus
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6602087
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76547
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Wed, 2 Mar 2011 06:56:46 +0000 (06:56 +0000)]
Use shim libvpx header to allow building with system libvpx.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6597077
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76509
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
phajdan.jr@chromium.org [Tue, 1 Mar 2011 18:02:17 +0000 (18:02 +0000)]
Define USE_SYSTEM_VPX pre-processor symbol when we're building with system vpx.
This is second part of http://codereview.chromium.org/6591069/
This is upstreaming Gentoo Linux changes.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/6598067
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76391
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Sat, 26 Feb 2011 02:13:53 +0000 (02:13 +0000)]
Initialize a variable, and revalidate indexes when necessary.
BUG=71788
TEST=none
Review URL: http://codereview.chromium.org/6598004
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76139
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
cevans@chromium.org [Sat, 26 Feb 2011 02:09:08 +0000 (02:09 +0000)]
Properly account the thread index on all exit paths from the function. Prevents
a crash with certain mp4 files.
BUG=73026
TEST=see bug
Review URL: http://codereview.chromium.org/6597005
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76138
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fischman@chromium.org [Fri, 25 Feb 2011 17:13:03 +0000 (17:13 +0000)]
Second attempt to prevent parse-error masking by ffmpeg.
First attempt was too aggressive, breaking decoding of files that were still
loading. This change instead looks for the specific invalid-data error and only
changes behavior for it.
TEST=arith.webm doesn't crash test_shell, and {browser,ui}_tests pass (and they broke pretty horribly on the first attempt, causing the DEPS rollback)
BUG=69218
Review URL: http://codereview.chromium.org/6532076
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@76055
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fischman@chromium.org [Thu, 24 Feb 2011 21:55:29 +0000 (21:55 +0000)]
Don't try to use vpx on arm. This reverts behavior back to pre-r75931, which
introduced an unintentional change to this behavior (because I fail at boolean
arithmetic).
TEST=none (no access to ARM machines, no ARM trybots)
BUG=70688
Review URL: http://codereview.chromium.org/6574039
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@75963
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fischman@chromium.org [Thu, 24 Feb 2011 19:54:44 +0000 (19:54 +0000)]
Use libvpx built from checked-in source instead of using pre-built libs, part 2.
This relies on issue 6542030 which must be checked in first.
It only changes the linux build.
BUG=70688
TEST=ffmpegsumo builds using libvpx source (an #error in a vpx source files breaks the build)
Review URL: http://codereview.chromium.org/6469064
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@75931
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fischman@chromium.org [Thu, 24 Feb 2011 19:44:13 +0000 (19:44 +0000)]
Revert 75440 - Stop masking errors (in ffmpeg) header parsing to avoid later crashes (in
ffmpeg) during video decoding.
Patch by fischman@chromium.org:
http://codereview.chromium.org/6534028/
BUG=69218
TEST=test_shell arith.web on the attachment from the bug no longer crashes.
TBR=scherkus@chromium.org
Review URL: http://codereview.chromium.org/6575039
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@75930
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
scherkus@chromium.org [Fri, 18 Feb 2011 21:19:52 +0000 (21:19 +0000)]
Stop masking errors (in ffmpeg) header parsing to avoid later crashes (in
ffmpeg) during video decoding.
Patch by fischman@chromium.org:
http://codereview.chromium.org/6534028/
BUG=69218
TEST=test_shell arith.web on the attachment from the bug no longer crashes.
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@75440
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
hbono@chromium.org [Tue, 15 Feb 2011 05:33:02 +0000 (05:33 +0000)]
Refactors ffmpeg.gyp.
We do not need magics for .asm files any longer since Evan fixed GYP Issue 102. This change just cleans up ffmpeg.gyp file to remove these workarounds.
BUG=none
TEST=build Chromium.
Review URL: http://codereview.chromium.org/6460023
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@74917
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
evan@chromium.org [Tue, 8 Feb 2011 22:19:20 +0000 (22:19 +0000)]
linux: don't warn about libavformat using its own deprecated APIs
Review URL: http://codereview.chromium.org/6461003
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@74188
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Fri, 28 Jan 2011 05:07:20 +0000 (05:07 +0000)]
remove patch 02 mov in ffmpeg and cleanup patch 14
BUG=71010
TESTED=media_bench.exe --stream=video security\*.mp4 still works
Review URL: http://codereview.chromium.org/6403001
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@72929
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
pkasting@chromium.org [Fri, 21 Jan 2011 22:11:24 +0000 (22:11 +0000)]
Update svn:ignores.
BUG=none
TEST=none
TBR=maruel
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@72206
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
tony@chromium.org [Wed, 19 Jan 2011 21:47:24 +0000 (21:47 +0000)]
Fix path to libvpx when building an upstream (webkit only) checkout.
Review URL: http://codereview.chromium.org/6360005
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@71842
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Mon, 17 Jan 2011 01:57:40 +0000 (01:57 +0000)]
fix div by zero in wav duration calculation.
this and vorbis have been upstreamed, so changing name/location as well.
build instructions updated to include wav
removed chrome build instructions, since chrome has its own readme
BUG=68910
TEST=ffmpeg -i addf8-GSM-GW.wav should use "Estimating duration from bitrate, this may be inaccurate"
Review URL: http://codereview.chromium.org/6300004
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@71582
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
mal@chromium.org [Sun, 16 Jan 2011 05:14:18 +0000 (05:14 +0000)]
Remove source/ from the path to the license file.
The DEPS have changed to pull from source/ directly and now the relative
path doesn't work with licenses.py.
BUG=None
TEST= ffmpeg shows up in about:credits again
R=fbarhcard,evan
Review URL: http://codereview.chromium.org/6370001
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@71569
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Tue, 11 Jan 2011 01:46:02 +0000 (01:46 +0000)]
minor cleanups of wav patch and vorbis patch
BUG=none
TEST=should still patch and build the same as before
Review URL: http://codereview.chromium.org/6177005
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@70984
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c
fbarchard@chromium.org [Mon, 10 Jan 2011 17:56:12 +0000 (17:56 +0000)]
ffmpeg patch updates for WAV duration and vorbis memory overflow.
WAV calculates duration based on header, not file size.
Vorbis checks for overflow before storing values in array.
BUG=68910
TEST=wav files should play with proper duration. Warning message should not appear about estimating duration based on file size.
Review URL: http://codereview.chromium.org/6162007
git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/ffmpeg/source@70905
4ff67af0-8c30-449e-8e8b-
ad334ec8d88c