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

Side by Side Diff: mojo/services/public/interfaces/native_viewport/native_viewport.mojom

Issue 617513003: Makes NativeViewport send OnBoundsChanged() after widget available (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « mojo/services/native_viewport/native_viewport_impl.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 import "mojo/services/gles2/command_buffer.mojom" 5 import "mojo/services/gles2/command_buffer.mojom"
6 import "mojo/services/public/interfaces/geometry/geometry.mojom" 6 import "mojo/services/public/interfaces/geometry/geometry.mojom"
7 import "mojo/services/public/interfaces/input_events/input_events.mojom" 7 import "mojo/services/public/interfaces/input_events/input_events.mojom"
8 import "mojo/services/public/interfaces/surfaces/surface_id.mojom" 8 import "mojo/services/public/interfaces/surfaces/surface_id.mojom"
9 9
10 module mojo { 10 module mojo {
11 11
12 [Client=NativeViewportClient] 12 [Client=NativeViewportClient]
13 interface NativeViewport { 13 interface NativeViewport {
14 // TODO(sky): having a create function is awkward. Should there be a factory 14 // TODO(sky): having a create function is awkward. Should there be a factory
15 // to create the NativeViewport that takes the size? 15 // to create the NativeViewport that takes the size?
16 // TODO(sky): callback should take size too.
17 Create(Size size) => (uint64 native_viewport_id); 16 Create(Size size) => (uint64 native_viewport_id);
18 Show(); 17 Show();
19 Hide(); 18 Hide();
20 Close(); 19 Close();
21 SetSize(Size size); 20 SetSize(Size size);
22 SubmittedFrame(SurfaceId surface_id); 21 SubmittedFrame(SurfaceId surface_id);
23 }; 22 };
24 23
25 interface NativeViewportClient { 24 interface NativeViewportClient {
25 // OnSizeChanged() is sent at least once after the callback from Create() is
26 // called.
26 OnSizeChanged(Size size); 27 OnSizeChanged(Size size);
27 OnDestroyed(); 28 OnDestroyed();
28 OnEvent(Event event) => (); 29 OnEvent(Event event) => ();
29 }; 30 };
30 31
31 // Connect to this interface before any other connections are made to configure 32 // Connect to this interface before any other connections are made to configure
32 // the NativeViewport service. 33 // the NativeViewport service.
33 interface NativeViewportConfig { 34 interface NativeViewportConfig {
34 // Call UseTestConfig() and block on the reply. This will ensure that the 35 // Call UseTestConfig() and block on the reply. This will ensure that the
35 // correct global initialization is done before subsequent connections. 36 // correct global initialization is done before subsequent connections.
36 UseTestConfig() => (); 37 UseTestConfig() => ();
37 38
38 // Call UseHeadlessConfig() and block on the reply. This will ensure that 39 // Call UseHeadlessConfig() and block on the reply. This will ensure that
39 // the native viewport is later created in headless mode. 40 // the native viewport is later created in headless mode.
40 UseHeadlessConfig() => (); 41 UseHeadlessConfig() => ();
41 }; 42 };
42 43
43 } 44 }
OLDNEW
« no previous file with comments | « mojo/services/native_viewport/native_viewport_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698