Support for YUV 4:4:4 subsampling in vpx decoder.

Support for YUV 4:4:4 subsampling in vpx decoder.

Depends on https://codereview.chromium.org/289373011/.

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

[scherkus (not reviewing), 2014-05-29 02:30:26]

can you get a VP9 444 copy of our bear video + a pipeline integration test
written?

https://code.google.com/p/chromium/codesearch#chromium/src/media/filters/pipe...

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

https://codereview.chromium.org/308543002/diff/1/media/filters/vpx_video_deco...
media/filters/vpx_video_decoder.cc:456: VideoFrame::Format codec_format;
nit: we typically try to initialize variables to some known value as the
compiler will do the right thing when optimizing the code

(if your conditions below ever get shuffled around, it's better to have known
values there in the first place so we don't end up with uninitialized values
down the road)

[sandersd (OOO until July 31), 2014-05-29 20:39:09]

I've added a test clip, but you should be aware that the bitstream format isn't
finalized and we will want to replace it later.

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

https://codereview.chromium.org/308543002/diff/1/media/filters/vpx_video_deco...
media/filters/vpx_video_decoder.cc:456: VideoFrame::Format codec_format;
On 2014/05/29 02:30:26, scherkus wrote:
> nit: we typically try to initialize variables to some known value as the
> compiler will do the right thing when optimizing the code
> 
> (if your conditions below ever get shuffled around, it's better to have known
> values there in the first place so we don't end up with uninitialized values
> down the road)

Done. I don't think that it's easier to read, but you are right that it's a
little safer.

[scherkus (not reviewing), 2014-05-29 20:42:33]

lgtm

you'll likely need to get someone to force-commit the .webm file as CQ doesn't
support binary files :\

[sandersd (OOO until July 31), 2014-06-03 21:08:57]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2014-06-03 21:10:29]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sandersd@chromium.org/308543002/40001

[sandersd (OOO until July 31), 2014-06-03 22:50:23]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2014-06-03 22:51:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sandersd@chromium.org/308543002/60001

[commit-bot: I haz the power, 2014-06-04 09:26:26]

Change committed as 274747