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

Unified Diff: components/exo/compositor_frame_sink_holder.h

Issue 2626293005: exo::CompositorFrameSink uses CompositorFrameSinkHolder* instead of mojo interface pointer (Closed)
Patch Set: Addressed nit Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/exo/compositor_frame_sink.cc ('k') | components/exo/compositor_frame_sink_holder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/exo/compositor_frame_sink_holder.h
diff --git a/components/exo/compositor_frame_sink_holder.h b/components/exo/compositor_frame_sink_holder.h
index 0f03d01aac4155f32e59f33a214d924306da37b3..7efa6ded119b91f1cc76217653c2dd18d821f787 100644
--- a/components/exo/compositor_frame_sink_holder.h
+++ b/components/exo/compositor_frame_sink_holder.h
@@ -31,10 +31,9 @@ class CompositorFrameSinkHolder
public cc::BeginFrameObserver,
public SurfaceObserver {
public:
- CompositorFrameSinkHolder(
- Surface* surface,
- std::unique_ptr<CompositorFrameSink> frame_sink,
- cc::mojom::MojoCompositorFrameSinkClientRequest request);
+ CompositorFrameSinkHolder(Surface* surface,
+ const cc::FrameSinkId& frame_sink_id,
+ cc::SurfaceManager* surface_manager);
bool HasReleaseCallbackForResource(cc::ResourceId id);
void SetResourceReleaseCallback(cc::ResourceId id,
@@ -84,7 +83,6 @@ class CompositorFrameSinkHolder
std::unique_ptr<cc::ExternalBeginFrameSource> begin_frame_source_;
bool needs_begin_frame_ = false;
cc::BeginFrameArgs last_begin_frame_args_;
- mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient> binding_;
base::WeakPtrFactory<CompositorFrameSinkHolder> weak_factory_;
« no previous file with comments | « components/exo/compositor_frame_sink.cc ('k') | components/exo/compositor_frame_sink_holder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698