Win Video Capture: add UYVY pixel format to Cr sink filter pin capabilities

Win Video Capture: add UYVY pixel format to Cr sink filter pin capabilities

This format is already supported in the rest of the capture path.
Bluefish 444 is the capture card using it.

BUG=508413
TEST=Capture using a Bluefish 444 capture card

Committed: https://crrev.com/6015405a94b144471eb0128779ad3c107192f445
Cr-Commit-Position: refs/heads/master@{#347861}

[mcasas, 2015-09-05 00:30:01]

Patchset #1 (id:1) has been deleted

[mcasas, 2015-09-05 01:30:16]

mcasas@chromium.org changed reviewers:
+ aspoon@gmail.com, emircan@chromium.org

[mcasas, 2015-09-05 01:30:16]

emircan@ PTAL

aspoon@ PTAL/FYI.

[aspoon, 2015-09-07 03:27:12]

On 2015/09/05 01:30:16, mcasas wrote:
> emircan@ PTAL
> 
> aspoon@ PTAL/FYI.

It's fine as is.  LGTM.

Just a quick (aside) question for the logging at the end of IsMediaTypeValid() -
why not do it the way
VideoCaptureDeviceWin::TranslateMediaSubtypeToPixelFormat() does it?
(https://code.google.com/p/chromium/codesearch#chromium/src/media/capture/vide...)

[mcasas, 2015-09-08 16:40:19]

On 2015/09/07 03:27:12, aspoon wrote:
> On 2015/09/05 01:30:16, mcasas wrote:
> > emircan@ PTAL
> > 
> > aspoon@ PTAL/FYI.
> 
> It's fine as is.  LGTM.
> 
> Just a quick (aside) question for the logging at the end of IsMediaTypeValid()
-
> why not do it the way
> VideoCaptureDeviceWin::TranslateMediaSubtypeToPixelFormat() does it?
>
(https://code.google.com/p/chromium/codesearch#chromium/src/media/capture/vide...)

Yeah, I wanted to make it simpler, but noticed
afterwards that some Fourccs are not printable.
Reused the StringFromGUID2() snippet.

[emircan, 2015-09-08 18:09:50]

lgtm % nit.

https://codereview.chromium.org/1325303004/diff/40001/media/capture/video/win...
File media/capture/video/win/sink_input_pin_win.cc (right):

https://codereview.chromium.org/1325303004/diff/40001/media/capture/video/win...
media/capture/video/win/sink_input_pin_win.cc:39: const VideoCaptureFormat&
SinkInputPin::ResultingFormat() {
We can inline and use accessor naming convention here.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Names

[mcasas, 2015-09-09 01:11:10]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2015-09-09 01:11:11]

The patchset sent to the CQ was uploaded after l-g-t-m from aspoon@gmail.com,
emircan@chromium.org
Link to the patchset: https://codereview.chromium.org/1325303004/#ps60001
(title: "emircan@ nit")

[commit-bot: I haz the power, 2015-09-09 01:11:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1325303004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1325303004/60001

[commit-bot: I haz the power, 2015-09-09 09:00:16]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-09-09 09:01:34]

Patchset 3 (id:??) landed as
https://crrev.com/6015405a94b144471eb0128779ad3c107192f445
Cr-Commit-Position: refs/heads/master@{#347861}

[mcasas, 2015-09-09 20:24:54]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/1332653005/ by mcasas@chromium.org.

The reason for reverting is: Temptative fix for

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester

timeout on 
WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest.MANUAL_TestVideoQuality/1

after 21330..