1 // Copyright 2015 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 blink.mojom;
6
7 // BoundingBox is defined with its upper-left point, width and height. It is
8 // essentially the same as DOMRect, but needs explicit conversion in blink.
9 struct BoundingBox {
Ken Rockot(use gerrit already)
2016/09/30 18:05:24
Note that ui/gfx/geometry/mojo/geometry.mojom alre
Note that ui/gfx/geometry/mojo/geometry.mojom already defines some standard
rectangle types (Rect for ints and RectF for floats.) I think we should avoid
introducing more in blink mojom.
You can
import "ui/gfx/geomoetry/mojo/geometry.mojom";
in this file and your message can return an array<gfx.mojom.RectF> instead.
The bindings target will then also need a public_deps on //ui/gfx/geometry/mojo.
On 2016/09/30 18:05:24, Ken Rockot wrote:
> Note that ui/gfx/geometry/mojo/geometry.mojom already defines some standard
> rectangle types (Rect for ints and RectF for floats.) I think we should avoid
> introducing more in blink mojom.
>
> You can
>
> import "ui/gfx/geomoetry/mojo/geometry.mojom";
>
> in this file and your message can return an array<gfx.mojom.RectF> instead.
>
> The bindings target will then also need a public_deps on
//ui/gfx/geometry/mojo.
Ken Rockot(use gerrit already)
2016/09/30 18:05:24
nit: Please document the format of frame_data. Spe
nit: Please document the format of frame_data. Specifically it's important to
convey the pixel format, the row/column ordering, and that the data is tightly
packed, since all of these things frequently vary in image buffer formats.
On 2016/09/30 18:05:24, Ken Rockot wrote:
> nit: Please document the format of frame_data. Specifically it's important to
> convey the pixel format, the row/column ordering, and that the data is tightly
> packed, since all of these things frequently vary in image buffer formats.
Issue 2369693002: Shapedetection module: Blink side implementation
(Closed)
Created 4 years, 2 months ago by xianglu
Modified 4 years, 2 months ago
Reviewers: esprehn, haraken, mcasas, Ken Rockot(use gerrit already), Tom Sepez
Base URL:
Comments: 68