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=177842

[Tom Finegan, 2013-01-18 21:47:48]

Re(re(re))landing the VP9 wrapper. Fingers crossed...

[commit-bot: I haz the power, 2013-01-18 21:50:52]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[xhwang, 2013-01-18 22:05:58]

lgtm since this is relanding the original patch. Based on offline conversation,
the diff since last landing are:

- phajdan@ removed libvpx.h from third_party/libvpx, so I had to update the
includes
- the ffmpeg roll landed, so I don't need the extra defines and casts in
ffmpeg_common
- Removed the libvpx DEPS roll from the CL (it's already landed separately)

https://codereview.chromium.org/12025030/diff/1/media/filters/vpx_video_decod...
File media/filters/vpx_video_decoder.cc (right):

https://codereview.chromium.org/12025030/diff/1/media/filters/vpx_video_decod...
media/filters/vpx_video_decoder.cc:249: if (buffer->GetDataSize()) {
scherkus@ is updating DecoderBuffer to put more restriction on it, your CL is
probably okay as is if you land before him. But double check it.

https://codereview.chromium.org/11993002/
https://codereview.chromium.org/11887011/

[Tom Finegan, 2013-01-18 22:15:23]

Trying the CQ again.

https://codereview.chromium.org/12025030/diff/1/media/filters/vpx_video_decod...
File media/filters/vpx_video_decoder.cc (right):

https://codereview.chromium.org/12025030/diff/1/media/filters/vpx_video_decod...
media/filters/vpx_video_decoder.cc:249: if (buffer->GetDataSize()) {
On 2013/01/18 22:05:59, xhwang wrote:
> scherkus@ is updating DecoderBuffer to put more restriction on it, your CL is
> probably okay as is if you land before him. But double check it.
> 
> https://codereview.chromium.org/11993002/
> https://codereview.chromium.org/11887011/

Thanks. I don't think the changes will cause a problem here.

[commit-bot: I haz the power, 2013-01-18 22:15:56]

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

[commit-bot: I haz the power, 2013-01-19 04:48:17]

Change committed as 177842