native_viewport support for ozone

mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.cc

Index: mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.cc
diff --git a/mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.cc b/mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7f8d16402bc1a2525c9acd993356137ca90fc4e3
--- /dev/null
+++ b/mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.cc
@@ -0,0 +1,78 @@
+// 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 "mojo/converters/ozone_drm_gpu/ozone_drm_gpu_type_converters.h"
+
+#include "mojo/converters/geometry/geometry_type_converters.h"
+
+namespace mojo {
+
+// static
+ui::DisplayMode_Params
+TypeConverter<ui::DisplayMode_Params, DisplayModePtr>::Convert(
+    const DisplayModePtr& in) {
+  auto out = ui::DisplayMode_Params();
+  out.size = in->size.To<gfx::Size>();
+  out.is_interlaced = in->is_interlaced;
+  out.refresh_rate = in->refresh_rate;
+  return out;
+}
+
+// static
+DisplayModePtr TypeConverter<DisplayModePtr, ui::DisplayMode_Params>::Convert(
+    const ui::DisplayMode_Params& in) {
+  auto out = DisplayMode::New();
+  out->size = Size::From<gfx::Size>(in.size);
+  out->is_interlaced = in.is_interlaced;
+  out->refresh_rate = in.refresh_rate;
+  return out.Pass();
+}
+
+// static
+ui::DisplaySnapshot_Params
+TypeConverter<ui::DisplaySnapshot_Params, DisplaySnapshotPtr>::Convert(
+    const DisplaySnapshotPtr& in) {
+  auto out = ui::DisplaySnapshot_Params();
+  out.display_id = in->display_id;
+  out.origin = in->origin.To<gfx::Point>();
+  out.physical_size = in->physical_size.To<gfx::Size>();
+  out.type = static_cast<ui::DisplayConnectionType>(in->type);

[jamesr, 2015/08/27 19:35:47]

is this type interpreted using the ui::DisplayConnectionType value on both ends?
if we wanted the mojom to be really general (i.e. be able to use a different
impl on the other end of the mojom interface) we should specify the possible
values in a mojo enum or with constants or something and then either
static_assert() that the values match and cast or use a switch to set values. i
don't think we need to do that in this patch necessarily but at least leave a
TODO to revisit

[cdotstout, 2015/08/27 21:28:33]

Done.

+  for (size_t i = 0; i < in->modes.size(); ++i) {
+    out.modes.push_back(in->modes[i].To<ui::DisplayMode_Params>());
+  }
+  out.has_current_mode = in->has_current_mode;
+  out.current_mode = in->current_mode.To<ui::DisplayMode_Params>();
+  out.has_native_mode = in->has_native_mode;
+  out.native_mode = in->native_mode.To<ui::DisplayMode_Params>();
+  out.product_id = in->product_id;
+  out.string_representation = in->string_representation;
+  return out;
+}
+
+// static
+DisplaySnapshotPtr
+TypeConverter<DisplaySnapshotPtr, ui::DisplaySnapshot_Params>::Convert(
+    const ui::DisplaySnapshot_Params& in) {
+  auto out = DisplaySnapshot::New();
+  out->display_id = in.display_id;
+  out->origin = Point::From<gfx::Point>(in.origin);
+  out->physical_size = Size::From<gfx::Size>(in.physical_size);
+  out->type = in.type;
+  auto modes = Array<DisplayModePtr>::New(in.modes.size());
+  for (size_t i = 0; i < in.modes.size(); ++i) {
+    auto mode = DisplayMode::From<ui::DisplayMode_Params>(in.modes[i]);
+    modes[i] = mode.Pass();
+  }
+  out->modes = modes.Pass();
+  out->has_current_mode = in.has_current_mode;
+  out->current_mode =
+      DisplayMode::From<ui::DisplayMode_Params>(in.current_mode);
+  out->has_native_mode = in.has_native_mode;
+  out->native_mode = DisplayMode::From<ui::DisplayMode_Params>(in.native_mode);
+  out->product_id = in.product_id;
+  out->string_representation = in.string_representation;
+  return out.Pass();
+}
+
+}  // namespace mojo