Win Video Capture: Add support for HDYC pixel format.

Win Video Capture: Add support for HDYC pixel format.

This pixel format is a synonym of YUYV and is needed
for at least the DeckLink video capture card.
This format is not specified in Windows headers
probably because this is just a synonym of other
pixel formats, IOW they are all the same except for
the first word, which is the name read in hex
backwards:
30323449 --> 0x30 0x32 0x34 0x49 -> 024I
43594448 --> 0x43 0x59 0x44 0x48 -> CYDH

and then all the YUV (422 and 420 alike) use the same
trailing sequence 
... 0x0000, 0x0010, {0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}}

HDYC: http://www.fourcc.org/yuv.php#HDYC

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

[mcasas, 2014-06-05 23:50:10]

niklase@ PTAL.

Note that you mentioned HDYC --> YUY2 but according to [1]
is actually UYVY... so we need to test this patch.

[1] http://www.fourcc.org/yuv.php#HDYC

[Niklas Enbom, 2014-06-06 16:52:27]

ok, will test first
On 2014/06/05 23:50:10, mcasas wrote:
> niklase@ PTAL.
> 
> Note that you mentioned HDYC --> YUY2 but according to [1]
> is actually UYVY... so we need to test this patch.
> 
> [1] http://www.fourcc.org/yuv.php#HDYC

[Niklas Enbom, 2014-06-09 20:32:32]

Doesn't solve all the problems, but a step in the right direction. lgtm
On 2014/06/06 16:52:27, Niklas Enbom wrote:
> ok, will test first
> On 2014/06/05 23:50:10, mcasas wrote:
> > niklase@ PTAL.
> > 
> > Note that you mentioned HDYC --> YUY2 but according to [1]
> > is actually UYVY... so we need to test this patch.
> > 
> > [1] http://www.fourcc.org/yuv.php#HDYC

[mcasas, 2014-06-09 22:08:13]

tommi@ PTAL.

[tommi (sloooow) - chröme, 2014-06-10 07:38:45]

lgtm

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
File media/video/capture/win/sink_filter_win.cc (right):

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
media/video/capture/win/sink_filter_win.cc:15: 0x30323449, 0x0000, 0x0010,
{0x80, 0x00, 0x00, 0xAA, 0x00, 0x38, 0x9B, 0x71}};
nit: I prefer the closing } on the next line as it was. It's more consistent
with how we initialize variables like this.

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
File media/video/capture/win/sink_filter_win.h (right):

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
media/video/capture/win/sink_filter_win.h:25: extern GUID kMediaSubTypeHDYC;
do you know why these aren't in the media namespace?

[mcasas, 2014-06-10 18:46:29]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2014-06-10 18:49:25]

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

[mcasas, 2014-06-10 18:56:53]

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
File media/video/capture/win/sink_filter_win.cc (right):

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
media/video/capture/win/sink_filter_win.cc:15: 0x30323449, 0x0000, 0x0010,
{0x80, 0x00, 0x00, 0xAA, 0x00, 0x38, 0x9B, 0x71}};
On 2014/06/10 07:38:45, tommi wrote:
> nit: I prefer the closing } on the next line as it was. It's more consistent
> with how we initialize variables like this.

Done.

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
File media/video/capture/win/sink_filter_win.h (right):

https://codereview.chromium.org/317073010/diff/20001/media/video/capture/win/...
media/video/capture/win/sink_filter_win.h:25: extern GUID kMediaSubTypeHDYC;
On 2014/06/10 07:38:45, tommi wrote:
> do you know why these aren't in the media namespace?

They shouldn't have to, so I moved them into media
namespace.

[commit-bot: I haz the power, 2014-06-11 04:52:43]

Change committed as 276260