MEDIASUBTYPE_RGB32 support for Windows

content/browser/renderer_host/media/video_capture_device_client.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/media/video_capture_device_client.h"
 
 #include "base/bind.h"
 #include "base/strings/stringprintf.h"
 #include "base/trace_event/trace_event.h"
 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
@@ skipping 145 matching lines @@
 #if defined(OS_WIN)
       // TODO(wjia): Currently, for RGB24 on WIN, capture device always
       // passes in positive src_width and src_height. Remove this hardcoded
       // value when nagative src_height is supported. The negative src_height
       // indicates that vertical flipping is needed.
       flip = true;
 #endif
       break;
     case media::PIXEL_FORMAT_ARGB:
       origin_colorspace = libyuv::FOURCC_ARGB;
+#if defined(OS_WIN)
+      flip = true;

[mcasas, 2015/03/27 01:02:11]

Please be sure that devices using PIXEL_FORMAT_ARGB, i.e.
those reporting MEDIASUBTYPE_ARGB32 in [1],  are OK with
this extra flipping.

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/media/video/captur...

[emircan, 2015/03/27 17:50:39]

I see your concern. In order to not change the existing workflow for
PIXEL_FORMAT_ARGB, I defined PIXEL_FORMAT_RGB32 with the same colorspace
conversion and flip.

+#endif
       break;
     case media::PIXEL_FORMAT_MJPEG:
       origin_colorspace = libyuv::FOURCC_MJPG;
       break;
     default:
       NOTREACHED();
   }
 
   // The input |length| can be greater than the required buffer size because of
   // paddings and/or alignments, but it cannot be smaller.
@@ skipping 181 matching lines @@
 }
 
 void VideoCaptureDeviceClient::OnLog(
     const std::string& message) {
   BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
                           base::Bind(&VideoCaptureController::DoLogOnIOThread,
                                      controller_, message));
 }
 
 }  // namespace content