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

Issue 10196006: Roll DEPS for FFmpeg merge. Shuffle testing. (Closed)

Created:
8 years, 8 months ago by DaleCurtis
Modified:
8 years, 8 months ago
CC:
chromium-reviews, feature-media-reviews_chromium.org
Visibility:
Public.

Description

Roll DEPS for FFmpeg merge. Shuffle testing. Lands the FFmpeg roll for M20. Manual test pass completed by vendors and myself. ASAN/Valgrind testing on Linux/Mac platforms on ia32/x64. TSAN testing on Windows, no new traces are expected. ffmpeg_revision: http://src.chromium.org/viewvc/chrome?view=rev&revision=133551 ffmpeg_hash: http://git.chromium.org/gitweb/?p=chromium/third_party/ffmpeg.git;a=commit;h=f8d71c394e071249f26703669baf1093009620d7 List of remaining diff's from upstream: http://pastebin.com/t5jTbvm9 BUG=49709, 124684 TEST=Video Test Matrix. unittests + asan/valgrind. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=133592

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -13 lines) Patch
M DEPS View 1 chunk +2 lines, -2 lines 0 comments Download
M media/ffmpeg/ffmpeg_regression_tests.cc View 4 chunks +19 lines, -6 lines 0 comments Download
M media/ffmpeg/ffmpeg_unittest.cc View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
DaleCurtis
PTAL. Soaking in the try bots now.
8 years, 8 months ago (2012-04-23 23:04:44 UTC) #1
scherkus (not reviewing)
8 years, 8 months ago (2012-04-24 01:32:09 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698