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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 9447029: Second attempt to land ffmpeg roll. (Closed)

Created:
8 years, 10 months ago by DaleCurtis
Modified:
8 years, 9 months ago
CC:
chromium-reviews, robertshield, amit, pam+watch_chromium.org, feature-media-reviews_chromium.org
Visibility:
Public.

Description

Second attempt to land ffmpeg roll. Attempt 2 at landing http://codereview.chromium.org/9317096/ Same as before except for fixes in checkperms/ However, fixes have landed elsewhere for: - mp3 decode issue. - FrameRateNoVsyncCanvasInternalTest.fishbowl/0 BUG=110776 TEST=unittests, layouttests, trybots, perf tests... Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124501

Patch Set 1 #

Patch Set 2 : Roll DEPS. #

Patch Set 3 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+158 lines, -126 lines) Patch
M DEPS View 1 2 3 chunks +7 lines, -5 lines 0 comments Download
M chrome/app/chrome.dll.deps View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/tools/build/win/FILES.cfg View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome_frame/test/perf/chrome_frame_perftest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/base/audio_decoder_config.h View 1 chunk +3 lines, -0 lines 0 comments Download
M media/base/media_posix.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/base/media_win.cc View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M media/ffmpeg/ffmpeg_common.h View 1 chunk +1 line, -0 lines 0 comments Download
M media/ffmpeg/ffmpeg_common.cc View 4 chunks +19 lines, -19 lines 0 comments Download
M media/ffmpeg/ffmpeg_regression_tests.cc View 1 2 5 chunks +34 lines, -17 lines 0 comments Download
M media/ffmpeg/ffmpeg_unittest.cc View 8 chunks +22 lines, -12 lines 0 comments Download
M media/ffmpeg/file_protocol.cc View 2 chunks +2 lines, -1 line 0 comments Download
M media/filters/audio_file_reader.cc View 5 chunks +8 lines, -8 lines 0 comments Download
M media/filters/ffmpeg_audio_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/ffmpeg_demuxer.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/ffmpeg_demuxer_unittest.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M media/filters/ffmpeg_glue.h View 2 chunks +17 lines, -17 lines 0 comments Download
M media/filters/ffmpeg_glue.cc View 4 chunks +3 lines, -3 lines 0 comments Download
M media/filters/ffmpeg_glue_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/ffmpeg_video_decoder.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/ffmpeg_video_decoder_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/test/ffmpeg_tests/ffmpeg_tests.cc View 6 chunks +7 lines, -8 lines 0 comments Download
M media/tools/media_bench/media_bench.cc View 7 chunks +8 lines, -9 lines 0 comments Download
M media/webm/webm_stream_parser.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M tools/checkperms/checkperms.py View 1 2 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
DaleCurtis
PTAL. Try's in progress.
8 years, 10 months ago (2012-02-24 19:27:39 UTC) #1
scherkus (not reviewing)
8 years, 10 months ago (2012-02-24 21:53:28 UTC) #2 
LGTM

Powered by Google App Engine
This is Rietveld 408576698