add more color formats NV21 and YV12 in video capture (mainly from Android).

add more color formats NV21 and YV12 in video capture (mainly from Android).
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=132808

[scherkus (not reviewing), 2012-04-13 03:01:57]

LGTM w/ q

http://codereview.chromium.org/10075001/diff/1/media/base/yuv_convert.h
File media/base/yuv_convert.h (right):

http://codereview.chromium.org/10075001/diff/1/media/base/yuv_convert.h#newco...
media/base/yuv_convert.h:116: void ConvertNV21ToYUV(const uint8* src,
do you know why none of these methods accept a stride?

[scherkus (not reviewing), 2012-04-13 03:02:15]

is there a BUG= or test for this?

[wjia(left Chromium), 2012-04-17 23:49:32]

There is no BUG # for this.

http://codereview.chromium.org/10075001/diff/1/media/base/yuv_convert.h
File media/base/yuv_convert.h (right):

http://codereview.chromium.org/10075001/diff/1/media/base/yuv_convert.h#newco...
media/base/yuv_convert.h:116: void ConvertNV21ToYUV(const uint8* src,
On 2012/04/13 03:01:57, scherkus wrote:
> do you know why none of these methods accept a stride?

There could be strides for Y and UV (typically aligned to 16 bytes). So far, we
haven't run into stride issue, possibly due to camera resolution selected. Will
re-visit when stride is needed. It requires some change in device capability.

[commit-bot: I haz the power, 2012-04-17 23:49:46]

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

[commit-bot: I haz the power, 2012-04-18 00:23:36]

Try job failure for 10075001-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...