16 bpp video stream capture, render and createImageBitmap(video) using (CPU) shared memory buffers

media/capture/video/win/video_capture_device_win.cc

Index: media/capture/video/win/video_capture_device_win.cc
diff --git a/media/capture/video/win/video_capture_device_win.cc b/media/capture/video/win/video_capture_device_win.cc
index af34f863ea766440382e732b96af0fcbd4acf2e2..9014876c1d956f5dbb3b39284b34255a953e2e59 100644
--- a/media/capture/video/win/video_capture_device_win.cc
+++ b/media/capture/video/win/video_capture_device_win.cc
@@ -16,7 +16,7 @@
 #include "base/win/scoped_co_mem.h"
 #include "base/win/scoped_variant.h"
 #include "media/base/timestamp_constants.h"
-#include "media/capture/video/blob_utils.h"
+#include "media/capture/video/video_capture_utils.h"
 
 using base::win::ScopedCoMem;
 using base::win::ScopedComPtr;
@@ -175,6 +175,9 @@ VideoCaptureDeviceWin::TranslateMediaSubtypeToPixelFormat(
     if (sub_type == pixel_format.sub_type)
       return pixel_format.format;
   }
+  if (IsY16FormatFourCc(sub_type.Data1)) {
+    return PIXEL_FORMAT_Y16;
+  }

[mcasas, 2016/10/21 00:10:51]

Here and in the equivalent MF version, I don't like
that you give special treatment to the Y16 formats.
We should do it more like the V4L2 delegate. I 
suggest: extend |kMediaSubtypeToPixelFormatCorrespondence|
with an entry for |Data1|. The currently existing
MEDIASUBTYPE_* won't have anything meaningful
there, and the Y16 will be listed individually, and
then you'll have a single for loop in l.174. Again, this
will also apply to video_capture_device_mf_win.cc .

[aleksandar.stojiljkovic, 2016/10/21 22:11:11]

No problem - I though it was a good idea to share the same code with
sink_input_pin_win.cc, but I'll just handle it the same way as kMediaSubTypeHDYC
and kMediaSubTypeI420 in [1]
https://cs.chromium.org/chromium/src/media/capture/video/win/sink_filter_win....

[aleksandar.stojiljkovic, 2016/10/24 14:57:49]

Done. Done like for other formats.

 #ifndef NDEBUG
   WCHAR guid_str[128];
   StringFromGUID2(sub_type, guid_str, arraysize(guid_str));