Add wrapper class to media for support of VP9 video, and add a command line flag to enable the supp…

Add wrapper class to media for support of VP9 video, and add a command line flag to enable the support.

This initial version of the wrapper provides support for decoding VP9 video in WebM container files, and is disabled by default.

New flag added: --enable-vp9-playback

TBR=brettw,scherkus,xhwang
BUG=166094
TEST=VP9 video in WebM containers plays back in <video> elements when --enable-vp9-playback is specified on the command line.


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=178799

[Tom Finegan, 2013-01-24 20:27:43]

Relanding VP9 support.

[scherkus (not reviewing), 2013-01-24 21:23:28]

lgtm

[brettw, 2013-01-24 21:32:13]

LGTM. I only looked at non-media/ stuff.

[xhwang, 2013-01-24 22:33:32]

lgtm % comment

https://codereview.chromium.org/12045060/diff/4007/media/filters/vpx_video_de...
File media/filters/vpx_video_decoder.cc (right):

https://codereview.chromium.org/12045060/diff/4007/media/filters/vpx_video_de...
media/filters/vpx_video_decoder.cc:267: DCHECK(video_frame);
Can we DCHECK(!buffer->IsEndOfStream) here? Just want to make sure we don't hit
this bug:

http://code.google.com/p/webm/issues/detail?id=524

[Tom Finegan, 2013-01-24 23:39:47]

https://codereview.chromium.org/12045060/diff/4007/media/filters/vpx_video_de...
File media/filters/vpx_video_decoder.cc (right):

https://codereview.chromium.org/12045060/diff/4007/media/filters/vpx_video_de...
media/filters/vpx_video_decoder.cc:267: DCHECK(video_frame);
On 2013/01/24 22:33:32, xhwang wrote:
> Can we DCHECK(!buffer->IsEndOfStream) here? Just want to make sure we don't
hit
> this bug:
> 
> http://code.google.com/p/webm/issues/detail?id=524

Done.

[commit-bot: I haz the power, 2013-01-25 00:56:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tomfinegan@chromium.org/12045060/5003

[commit-bot: I haz the power, 2013-01-25 03:31:20]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-01-25 04:11:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tomfinegan@chromium.org/12045060/5003

[commit-bot: I haz the power, 2013-01-25 09:26:18]

Change committed as 178799