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

Unified Diff: components/exo/compositor_frame_sink_holder.cc

Issue 2691453002: [cc] Track observer status in ExternalBeginFrameSource. (Closed)
Patch Set: Fix comments. Created 3 years, 10 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
Index: components/exo/compositor_frame_sink_holder.cc
diff --git a/components/exo/compositor_frame_sink_holder.cc b/components/exo/compositor_frame_sink_holder.cc
index df76d11728662eb9edd013eecc625f08d04c934f..11e0b5ea8364094e5f8435f9ee4c06d657c4b967 100644
--- a/components/exo/compositor_frame_sink_holder.cc
+++ b/components/exo/compositor_frame_sink_holder.cc
@@ -58,6 +58,7 @@ void CompositorFrameSinkHolder::DidReceiveCompositorFrameAck() {
}
void CompositorFrameSinkHolder::OnBeginFrame(const cc::BeginFrameArgs& args) {
+ // TODO(eseckler): Hook up |surface_| to the ExternalBeginFrameSource.
if (surface_)
surface_->BeginFrame(args.frame_time);
@@ -102,6 +103,10 @@ void CompositorFrameSinkHolder::OnNeedsBeginFrames(bool needs_begin_frames) {
frame_sink_->SetNeedsBeginFrame(needs_begin_frames);
}
+void CompositorFrameSinkHolder::OnDidFinishFrame(const cc::BeginFrameAck& ack) {
+ // TODO(eseckler): Pass on the ack to frame_sink_.
+}
+
////////////////////////////////////////////////////////////////////////////////
// SurfaceObserver overrides:
« no previous file with comments | « components/exo/compositor_frame_sink_holder.h ('k') | content/renderer/gpu/compositor_external_begin_frame_source.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698