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

Side by Side Diff: cc/ipc/mojo_compositor_frame_sink.mojom

Issue 2493223002: Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink (Closed)
Patch Set: Addressed comments: Created 4 years 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 module cc.mojom; 5 module cc.mojom;
6 6
7 import "cc/ipc/begin_frame_args.mojom"; 7 import "cc/ipc/begin_frame_args.mojom";
8 import "cc/ipc/compositor_frame.mojom"; 8 import "cc/ipc/compositor_frame.mojom";
9 import "cc/ipc/frame_sink_id.mojom"; 9 import "cc/ipc/frame_sink_id.mojom";
10 import "cc/ipc/local_frame_id.mojom"; 10 import "cc/ipc/local_frame_id.mojom";
(...skipping 28 matching lines...) Expand all
39 // Removes surface references. 39 // Removes surface references.
40 RemoveSurfaceReferences(array<SurfaceReference> references); 40 RemoveSurfaceReferences(array<SurfaceReference> references);
41 41
42 // Notify that the surface is no longer in use (and is okay to be evicted) so 42 // Notify that the surface is no longer in use (and is okay to be evicted) so
43 // that its resources gets returned in time. 43 // that its resources gets returned in time.
44 EvictFrame(); 44 EvictFrame();
45 45
46 // TODO(staraz): Delete Require() and Satisfy() once surface references 46 // TODO(staraz): Delete Require() and Satisfy() once surface references
47 // (CL 2541683004) are ready. 47 // (CL 2541683004) are ready.
48 // Add the provided |sequence| as a destruction dependency of the 48 // Add the provided |sequence| as a destruction dependency of the
49 // surface associated with the provided |local_frame_id|. 49 // surface associated with the provided |local_frame_id|.addressed comments
Fady Samuel 2016/12/07 20:13:00 nit: Remove "addressed comments"
Alex Z. 2016/12/07 21:13:43 Done.
50 Require(cc.mojom.LocalFrameId local_frame_id, 50 Require(cc.mojom.LocalFrameId local_frame_id,
51 cc.mojom.SurfaceSequence sequence); 51 cc.mojom.SurfaceSequence sequence);
52 52
53 // Mark the sequence as satisfied and garbage collect surfaces. 53 // Mark the sequence as satisfied and garbage collect surfaces.
54 Satisfy(cc.mojom.SurfaceSequence sequence); 54 Satisfy(cc.mojom.SurfaceSequence sequence);
55 55
56 // TODO(fsamuel): ReadbackBitmap API would be useful here. 56 // TODO(fsamuel): ReadbackBitmap API would be useful here.
57 }; 57 };
58 58
59 interface MojoCompositorFrameSinkClient { 59 interface MojoCompositorFrameSinkClient {
(...skipping 17 matching lines...) Expand all
77 }; 77 };
78 78
79 // MojoCompositorFrameSinkPrivate is used by the display compositor host to set 79 // MojoCompositorFrameSinkPrivate is used by the display compositor host to set
80 // up BeginFrameSource hierarchies. This API lives in SurfaceManager but is 80 // up BeginFrameSource hierarchies. This API lives in SurfaceManager but is
81 // called from here in order to ensure that hierarchy registration does not race 81 // called from here in order to ensure that hierarchy registration does not race
82 // CompositorFrameSink creation. 82 // CompositorFrameSink creation.
83 interface MojoCompositorFrameSinkPrivate { 83 interface MojoCompositorFrameSinkPrivate {
84 AddChildFrameSink(FrameSinkId child_frame_sink_id); 84 AddChildFrameSink(FrameSinkId child_frame_sink_id);
85 RemoveChildFrameSink(FrameSinkId child_frame_sink_id); 85 RemoveChildFrameSink(FrameSinkId child_frame_sink_id);
86 }; 86 };
OLDNEW
« no previous file with comments | « ash/test/ash_test_helper.cc ('k') | components/exo/BUILD.gn » ('j') | components/exo/surface.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698