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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 3005034: deps roll for new ffmpeg and switch to av_register_protocol2... (Closed)

Created:
10 years, 5 months ago by fbarchard
Modified:
9 years, 7 months ago
CC:
chromium-reviews, scherkus (not reviewing), pam+watch_chromium.org, awong, Alpha Left Google
Visibility:
Public.

Description

deps roll for new ffmpeg and switch to av_register_protocol2 BUG=48037 TEST=layout tests should still pass for media Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53885

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -11 lines) Patch
M DEPS View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/base/mock_ffmpeg.h View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/base/mock_ffmpeg.cc View 1 2 3 chunks +4 lines, -4 lines 0 comments Download
M media/ffmpeg/ffmpeg_common.h View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M media/ffmpeg/ffmpeg_unittest.cc View 2 1 chunk +1 line, -1 line 0 comments Download
M media/filters/ffmpeg_glue.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/test/ffmpeg_tests/ffmpeg_tests.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/tools/media_bench/media_bench.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M media/tools/omx_test/omx_test.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
fbarchard
Roll to ffmpeg that has Alpha's vp3 patch to seek to 0.
10 years, 5 months ago (2010-07-27 16:58:50 UTC) #1
scherkus (not reviewing)
You'll have to include the change to media/ffmpeg/ffmpeg_unittest.cc now
10 years, 5 months ago (2010-07-27 20:51:29 UTC) #2
fbarchard
CL53860 for oggdec patch ffmpeg_tests use av_register_protocol2
10 years, 5 months ago (2010-07-27 23:24:06 UTC) #3
fbarchard
correction: ffmpeg_unittests use av_register_protocol2 Current status: [ PASSED ] 120 tests. [ FAILED ] 9 ...
10 years, 5 months ago (2010-07-27 23:29:28 UTC) #4
fbarchard
TBR=scherkus
10 years, 5 months ago (2010-07-28 00:54:53 UTC) #5
scherkus (not reviewing)
10 years, 5 months ago (2010-07-28 01:20:40 UTC) #6 
As discussed over IM/email this is LGTM

Powered by Google App Engine
This is Rietveld 408576698