Check VideoFrame::IsValidConfig() prior to allocating buffers for FFmpeg.

Check VideoFrame::IsValidConfig() prior to allocating buffers for FFmpeg.

BUG=140085
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150215

[scherkus (not reviewing), 2012-08-06 20:42:34]

rbultje: sanity check
fischman: OWNERS + review

https://chromiumcodereview.appspot.com/10827149/diff/1/media/filters/ffmpeg_v...
File media/filters/ffmpeg_video_decoder.cc (right):

https://chromiumcodereview.appspot.com/10827149/diff/1/media/filters/ffmpeg_v...
media/filters/ffmpeg_video_decoder.cc:88: VideoFrame::CreateFrame(format, width,
height, kNoTimestamp());
this DCHECKs for IsValidConfig() but in release mode we'll end up creating
invalid frames

[Ami GONE FROM CHROMIUM, 2012-08-06 21:14:19]

lgtm

[rbultje1, 2012-08-06 21:24:46]

lgtm also.

[commit-bot: I haz the power, 2012-08-06 22:16:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scherkus@chromium.org/10827149/1

[commit-bot: I haz the power, 2012-08-06 22:42:27]

Try job failure for 10827149-1 (retry) on mac_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-08-06 23:31:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scherkus@chromium.org/10827149/9003

[commit-bot: I haz the power, 2012-08-07 01:21:32]

Change committed as 150215