native_viewport support for ozone

mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom

+// 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.
+
+module mojo;
+
+import "geometry/public/interfaces/geometry.mojom";
+
+struct DisplayMode {
+  Size size;
+  bool is_interlaced;
+  float refresh_rate;
+};
+
+struct DisplaySnapshot {
+  int64 display_id;
+  Point origin;
+  Size physical_size;
+  int32 type;

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

is this an enum?

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

Done.

+  bool has_current_mode;

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

could we put some basic documentation here? most of these fields are pretty
self-evident, but i'm not sure what 'has_current_mode' means or what the
relationship of display_id / product_id is

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

Added some docs.  Michael please advise if I didn't get it right.

[spang, 2015/08/28 19:38:20]

display_id identifies a display attached to the system. We try to make it stable
across reboots.

product_id is a manufacturer provided id for the hardware model, not unique per
unit

+  bool has_native_mode;
+  array<DisplayMode> modes;
+  DisplayMode current_mode;
+  DisplayMode native_mode;

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

are current / native always members of |modes|? maybe these could be indices
instead?

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

I think so.  Michael, any reason not to make the change in chromium as well?

Added TODO for now.

[spang, 2015/08/28 19:38:19]

I think so as well.

+  int64 product_id;
+  string string_representation;
+};
+
+interface OzoneDrmGpu {
+  AddGraphicsDevice(string file_path, int32 file_descriptor);
+  CreateWindow(int64 widget);
+  WindowBoundsChanged(int64 widget, Rect bounds);
+
+  RefreshNativeDisplays();
+  ConfigureNativeDisplay(int64 id, DisplayMode mode, Point originhost);
+};