[Mojo Video Capture] Add test ReceiveFramesFromFakeCaptureDevice

services/video_capture/device_factory_media_to_mojo_adapter.cc

Index: services/video_capture/device_factory_media_to_mojo_adapter.cc
diff --git a/services/video_capture/device_factory_media_to_mojo_adapter.cc b/services/video_capture/device_factory_media_to_mojo_adapter.cc
index c39f5c90743dc4b026daad77763c847242d048a4..e15a69c188cdc276f19fd61c4ae1106f6d5f6ba1 100644
--- a/services/video_capture/device_factory_media_to_mojo_adapter.cc
+++ b/services/video_capture/device_factory_media_to_mojo_adapter.cc
@@ -48,7 +48,22 @@ void DeviceFactoryMediaToMojoAdapter::GetSupportedFormats(
     const media::VideoCaptureDeviceDescriptor& device_descriptor,
     const GetSupportedFormatsCallback& callback) {
   std::vector<VideoCaptureFormat> result;
-  NOTIMPLEMENTED();
+  std::vector<media::VideoCaptureFormat> media_formats;
+  device_factory_->GetSupportedFormats(device_descriptor, &media_formats);
+  for (const auto& media_format : media_formats) {
+    // The Video Capture Service requires devices to deliver frames either in
+    // I420 or MJPEG formats.
+    if (media_format.pixel_format != media::PIXEL_FORMAT_I420 &&
+        media_format.pixel_format != media::PIXEL_FORMAT_MJPEG)

[mcasas, 2016/11/09 23:50:29]

Y16 was added very recently to FakeVCD and Win [1]
to support depth-cameras, perhaps you should add 
it here as well ?

[1]
https://cs.chromium.org/search/?q=PIXEL_FORMAT_Y16+media/capture/video&sq=pac...

[chfremer, 2016/11/10 00:28:48]

Thanks for the heads up. 
I have already seen and merged in some of the Y16 code changes.
For the Mojo service, please allow me to defer adding the Y16 until we get
closer to the point where we are actually going to connect it to
VideoCaptureController. I will add a work item in the design doc to ensure we
add the support for Y16 before we connect it.

[mcasas, 2016/11/10 01:21:25]

Alright. If it's in a few places (ideally, one), just add a 
TODO to link to the bug 624436.

[chfremer, 2016/11/10 04:41:35]

Done.

+      continue;
+    VideoCaptureFormat format;
+    format.frame_size = media_format.frame_size;
+    format.frame_rate = media_format.frame_rate;
+    if (std::find(std::begin(result), std::end(result), format) !=
+        std::end(result))
+      continue;  // Result already contains this format

[mcasas, 2016/11/09 23:50:29]

{} because the if() condition expands > 1 line.

[chfremer, 2016/11/10 00:28:48]

Done.

+    result.push_back(format);
+  }
   callback.Run(std::move(result));
 }