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

Unified Diff: components/viz/hittest/public/interfaces/hittest_data.mojom

Issue 2908783002: WIP Hittest Component.
Patch Set: improvements from reviewer comments Created 3 years, 6 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: components/viz/hittest/public/interfaces/hittest_data.mojom
diff --git a/components/viz/hittest/public/interfaces/hittest_data.mojom b/components/viz/hittest/public/interfaces/hittest_data.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..21537af0f00853102cb4e2b61d8590b6c43d1188
--- /dev/null
+++ b/components/viz/hittest/public/interfaces/hittest_data.mojom
@@ -0,0 +1,39 @@
+// Copyright 2017 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 viz.hittest.mojom;
+
+import "cc/ipc/surface_id.mojom";
+import "ui/gfx/geometry/mojo/geometry.mojom";
+import "ui/gfx/mojo/transform.mojom";
+
+enum HittestRegionFlags {
+
+ // embedded child region mapped to a surface
rjkroege 2017/06/07 17:16:46 Sentence comments.
gklassen 2017/06/07 20:04:45 Done.
+ HITTEST_SURFACE = 0,
+
+ // irregular boundary - send HittestRequest to resolve
+ HITTEST_ASK,
+
+ // no touch event handler exists - no need to send touch events
+ HITTEST_NO_TOUCH_EVENT_HANDLER,
+
+ // region ignored for hit testing ( eg. hover:none )
+ HITTEST_IGNORE
+};
+
+struct HittestRegion {
rjkroege 2017/06/07 17:16:46 lots of comments here.
gklassen 2017/06/07 20:04:45 Done.
+ cc.mojom.SurfaceId surface_id_;
+ HittestRegionFlags flags_;
+ gfx.mojom.Rect rect_;
+ gfx.mojom.Transform transform_;
+};
+
+struct HittestData {
+ cc.mojom.SurfaceId surface_id_;
+ gfx.mojom.Rect rect_;
+ HittestRegionFlags flags_;
rjkroege 2017/06/07 17:16:46 Should have different flag types.
gklassen 2017/06/07 20:04:44 Done.
+ gfx.mojom.Transform transform_;
rjkroege 2017/06/07 17:16:46 we convinced ourselves that it's not needed.
gklassen 2017/06/07 20:04:44 Done.
+ array<HittestRegion> regions_; // in front to back z-order
+};

Powered by Google App Engine
This is Rietveld 408576698