Use mojo for cursor control in ozone drm

ui/ozone/public/interfaces/cursor.mojom

+// 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.
+
+module ui.ozone.mojom;
+
+import "skia/public/interfaces/bitmap_array.mojom";
+import "ui/gfx/geometry/mojo/geometry.mojom";
+import "ui/gfx/mojo/accelerated_widget.mojom";
+
+// The Cursor setting interface provided by a Mojo-enabled Ozone platform where
+// functionality equivalent to the DrmCursorProxy is exposed by a dedicated thread

[Tom Sepez, 2016/07/19 16:14:38]

nit: 80 cols, also line 15

[rjkroege, 2016/07/21 01:10:42]

Done. I naïvely assumed presubmit would check .mojom. Filed a bug.

+// within the Ozone platform.
+interface Cursor {

[sky, 2016/07/19 13:21:48]

Can this be named CursorOzone? I realize that is implied by the namespace, but
it makes for less confusion when the same code ends up using two types of
Cursors.

[rjkroege, 2016/07/21 01:10:42]

Having wished for the same thing for bunch of other classes (e.g. Surface,
Service, Display, DisplayMode), I'm sympathetic. But, it will also be ugly for
an interface that is intended to only be used within ozone code to have the
extra Ozone in the name. I renamed so at least it's unique. WDYT?

+        // Sets the cursor |bitmaps| on |window| at |point| with |frame_delay_ms|.

[sky, 2016/07/19 13:21:48]

nit: 2 spaces? I try to format .mojom files as I would C++ files.

[rjkroege, 2016/07/21 01:10:42]

Done.

+        SetCursor(gfx.mojom.AcceleratedWidget window,
+                         skia.mojom.BitmapArray bitmaps,
+                         gfx.mojom.Point point,
+                          uint32 frame_delay_ms);
+
+        // Moves the cursor in |window| to |point|.
+        MoveCursor(gfx.mojom.AcceleratedWidget window, gfx.mojom.Point point);
+};
+