Video Capture Service: typemap video_capture's and media's VideoCaptureFormat

services/video_capture/public/interfaces/video_capture_format_traits.cc

Index: services/video_capture/public/interfaces/video_capture_format_traits.cc
diff --git a/services/video_capture/public/interfaces/video_capture_format_traits.cc b/services/video_capture/public/interfaces/video_capture_format_traits.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f37f4689b11a6b0efabafad6af7058acc584560e
--- /dev/null
+++ b/services/video_capture/public/interfaces/video_capture_format_traits.cc
@@ -0,0 +1,87 @@
+// Copyright 2016 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 "services/video_capture/public/interfaces/video_capture_format_traits.h"
+
+#include "ui/gfx/geometry/mojo/geometry.mojom.h"
+#include "ui/gfx/geometry/mojo/geometry_struct_traits.h"
+
+namespace mojo {
+
+// static
+const gfx::Size& StructTraits<video_capture::mojom::VideoCaptureFormatDataView,
+                              media::VideoCaptureFormat>::
+    frame_size(const media::VideoCaptureFormat& format) {
+  return format.frame_size;

[dcheng, 2016/10/04 22:09:26]

Inline, since we're just returning a member reference now =)

[mcasas, 2016/10/04 22:16:56]

Done.

+}
+
+// static
+media::mojom::VideoFormat StructTraits<
+    video_capture::mojom::VideoCaptureFormatDataView,
+    media::VideoCaptureFormat>::pixel_format(const media::VideoCaptureFormat&
+                                                 format) {
+  return static_cast<media::mojom::VideoFormat>(format.pixel_format);

[dcheng, 2016/10/04 22:09:26]

Ditto, inline these as well.

[mcasas, 2016/10/04 22:16:56]

Done.

+}
+
+// static
+video_capture::mojom::VideoPixelStorage StructTraits<
+    video_capture::mojom::VideoCaptureFormatDataView,
+    media::VideoCaptureFormat>::pixel_storage(const media::VideoCaptureFormat&
+                                                  format) {
+  return static_cast<video_capture::mojom::VideoPixelStorage>(
+      format.pixel_storage);
+}
+
+// static
+bool StructTraits<video_capture::mojom::VideoCaptureFormatDataView,
+                  media::VideoCaptureFormat>::
+    Read(video_capture::mojom::VideoCaptureFormatDataView data,
+         media::VideoCaptureFormat* out) {
+  if (!data.ReadFrameSize(&out->frame_size))
+    return false;
+  out->frame_rate = data.frame_rate();
+
+  // Since there are static_asserts in place in
+  // media/mojo/common/media_type_converters.cc to guarantee equality of the
+  // underlying representations, we can simply static_cast to convert.
+  // TODO(mcasas): Use EnumTraits for VideoPixelFormat, https://crbug.com/651897
+  out->pixel_format =
+      static_cast<media::VideoPixelFormat>(data.pixel_format());
+  if (!data.ReadPixelStorage(&out->pixel_storage))
+    return false;
+  return true;
+}
+
+// static
+video_capture::mojom::VideoPixelStorage
+EnumTraits<video_capture::mojom::VideoPixelStorage, media::VideoPixelStorage>::
+    ToMojom(media::VideoPixelStorage video_pixel_storage) {
+  switch (video_pixel_storage) {
+    case media::PIXEL_STORAGE_CPU:
+      return video_capture::mojom::VideoPixelStorage::CPU;
+    case media::PIXEL_STORAGE_GPUMEMORYBUFFER:
+      return video_capture::mojom::VideoPixelStorage::GPUMEMORYBUFFER;
+  }
+  NOTREACHED();
+  return video_capture::mojom::VideoPixelStorage::CPU;
+}
+
+// static
+bool EnumTraits<video_capture::mojom::VideoPixelStorage,
+                media::VideoPixelStorage>::
+    FromMojom(video_capture::mojom::VideoPixelStorage input,
+              media::VideoPixelStorage* out) {
+  switch (input) {
+    case video_capture::mojom::VideoPixelStorage::CPU:
+      *out = media::PIXEL_STORAGE_CPU;
+      return true;
+    case video_capture::mojom::VideoPixelStorage::GPUMEMORYBUFFER:
+      *out = media::PIXEL_STORAGE_GPUMEMORYBUFFER;
+      return true;
+  }
+  NOTREACHED();
+  return false;
+}
+
+}  // namespace mojo