Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(626)

Unified Diff: mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom

Issue 1309273005: native_viewport support for ozone (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: addressing review feedback Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom
diff --git a/mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom b/mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..bdaf0df7be6779f635211d752e077e99f930e5e7
--- /dev/null
+++ b/mojo/services/ozone_drm_gpu/public/interfaces/ozone_drm_gpu.mojom
@@ -0,0 +1,36 @@
+// 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 o
cdotstout 2015/08/27 21:28:33 Added some docs. Michael please advise if I didn'
spang 2015/08/28 19:38:20 display_id identifies a display attached to the sy
+ 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|? ma
cdotstout 2015/08/27 21:28:33 I think so. Michael, any reason not to make the c
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);
+};

Powered by Google App Engine
This is Rietveld 408576698