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

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

Issue 607233002: Nukes NativeViewportClient::OnCreated (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix build 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
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 Create(Size size); 14 // TODO(sky): having a create function is awkward. Should there be a factory
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);
15 Show(); 18 Show();
16 Hide(); 19 Hide();
17 Close(); 20 Close();
18 SetBounds(Size size); 21 SetSize(Size size);
19 SubmittedFrame(SurfaceId surface_id); 22 SubmittedFrame(SurfaceId surface_id);
20 }; 23 };
21 24
22 interface NativeViewportClient { 25 interface NativeViewportClient {
23 OnCreated(uint64 native_viewport_id); 26 OnSizeChanged(Size size);
24 OnBoundsChanged(Size size);
25 OnDestroyed(); 27 OnDestroyed();
26 OnEvent(Event event) => (); 28 OnEvent(Event event) => ();
27 }; 29 };
28 30
29 // Connect to this interface before any other connections are made to configure 31 // Connect to this interface before any other connections are made to configure
30 // the NativeViewport service. 32 // the NativeViewport service.
31 interface NativeViewportConfig { 33 interface NativeViewportConfig {
32 // Call UseTestConfig() and block on the reply. This will ensure that the 34 // Call UseTestConfig() and block on the reply. This will ensure that the
33 // correct global initialization is done before subsequent connections. 35 // correct global initialization is done before subsequent connections.
34 UseTestConfig() => (); 36 UseTestConfig() => ();
35 37
36 // Call UseHeadlessConfig() and block on the reply. This will ensure that 38 // Call UseHeadlessConfig() and block on the reply. This will ensure that
37 // the native viewport is later created in headless mode. 39 // the native viewport is later created in headless mode.
38 UseHeadlessConfig() => (); 40 UseHeadlessConfig() => ();
39 }; 41 };
40 42
41 } 43 }
OLDNEW
« no previous file with comments | « mojo/services/native_viewport/native_viewport_impl.cc ('k') | mojo/services/view_manager/display_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698