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

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

Issue 685013002: Refactors event dispatching of NativeViewport into its own interface (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: cleanup Created 6 years, 1 month 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/public/interfaces/geometry/geometry.mojom"; 5 import "mojo/services/public/interfaces/geometry/geometry.mojom";
6 import "mojo/services/public/interfaces/gpu/command_buffer.mojom"; 6 import "mojo/services/public/interfaces/gpu/command_buffer.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 Create(Size size) => (uint64 native_viewport_id); 16 Create(Size size) => (uint64 native_viewport_id);
17 Show(); 17 Show();
18 Hide(); 18 Hide();
19 Close(); 19 Close();
20 SetSize(Size size); 20 SetSize(Size size);
21 SubmittedFrame(SurfaceId surface_id); 21 SubmittedFrame(SurfaceId surface_id);
22 SetEventDispatcher(NativeViewportEventDispatcher dispatcher);
23 };
24
25 interface NativeViewportEventDispatcher {
26 OnEvent(Event event) => ();
22 }; 27 };
23 28
24 interface NativeViewportClient { 29 interface NativeViewportClient {
25 // OnSizeChanged() is sent at least once after the callback from Create() is 30 // OnSizeChanged() is sent at least once after the callback from Create() is
26 // called. 31 // called.
27 OnSizeChanged(Size size); 32 OnSizeChanged(Size size);
28 OnDestroyed(); 33 OnDestroyed();
29 OnEvent(Event event) => ();
30 }; 34 };
31 35
32 } 36 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698