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

Side by Side 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 module viz.hittest.mojom;
6
7 import "cc/ipc/surface_id.mojom";
8 import "ui/gfx/geometry/mojo/geometry.mojom";
9 import "ui/gfx/mojo/transform.mojom";
10
11 enum HittestRegionFlags {
12
13 // embedded child region mapped to a surface
rjkroege 2017/06/07 17:16:46 Sentence comments.
gklassen 2017/06/07 20:04:45 Done.
14 HITTEST_SURFACE = 0,
15
16 // irregular boundary - send HittestRequest to resolve
17 HITTEST_ASK,
18
19 // no touch event handler exists - no need to send touch events
20 HITTEST_NO_TOUCH_EVENT_HANDLER,
21
22 // region ignored for hit testing ( eg. hover:none )
23 HITTEST_IGNORE
24 };
25
26 struct HittestRegion {
rjkroege 2017/06/07 17:16:46 lots of comments here.
gklassen 2017/06/07 20:04:45 Done.
27 cc.mojom.SurfaceId surface_id_;
28 HittestRegionFlags flags_;
29 gfx.mojom.Rect rect_;
30 gfx.mojom.Transform transform_;
31 };
32
33 struct HittestData {
34 cc.mojom.SurfaceId surface_id_;
35 gfx.mojom.Rect rect_;
36 HittestRegionFlags flags_;
rjkroege 2017/06/07 17:16:46 Should have different flag types.
gklassen 2017/06/07 20:04:44 Done.
37 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.
38 array<HittestRegion> regions_; // in front to back z-order
39 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698