Make OffscreenCanvas animation in sync with its placeholder canvas's parent frame rate

content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc

Index: content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc
diff --git a/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc b/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc
index abd15c636fde412a497e1a6f02de96d7170332ca..64af271bcdd430f73789c14c4c311d6f502b3ae5 100644
--- a/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc
+++ b/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.cc
@@ -7,6 +7,7 @@
 #include "base/memory/ptr_util.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h"
+#include "content/browser/renderer_host/offscreen_canvas_surface_manager.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace content {
@@ -29,6 +30,9 @@ void OffscreenCanvasCompositorFrameSinkProviderImpl::CreateCompositorFrameSink(
   compositor_frame_sinks_[frame_sink_id] =
       base::MakeUnique<OffscreenCanvasCompositorFrameSink>(
           this, frame_sink_id, std::move(request), std::move(client));
+
+  OffscreenCanvasSurfaceManager::GetInstance()->RegisterFrameSinkToParent(
+      frame_sink_id);
 }
 
 cc::SurfaceManager*
@@ -42,4 +46,10 @@ void OffscreenCanvasCompositorFrameSinkProviderImpl::
   compositor_frame_sinks_.erase(frame_sink_id);
 }
 
+void OffscreenCanvasCompositorFrameSinkProviderImpl::
+    OnCompositorFrameSinkClientDestroyed(const cc::FrameSinkId& frame_sink_id) {
+  OffscreenCanvasSurfaceManager::GetInstance()->UnregisterFrameSinkFromParent(
+      frame_sink_id);

[Fady Samuel, 2017/01/19 17:10:53]

I don't think this is necessary. compositor_frame_sinks_ owns the
CompositorFrameSink.

I think you can simply put this line just below
compositor_frame_sinks_.erase(frame_sink_id); in
OffscreenCanvasCompositorFrmaeSinkProviderImpl::OnCompositorFrameSinkClientConnectionLost.

[xlai (Olivia), 2017/01/19 19:14:34]

I just double check, OnConnectionLost is not fired when I close the window.

The current situation is that OnCompositorFrameSinkClientConnectionLost is not
fired before SurfaceManager is destroyed, resulting in an assert failure in
DCHECK_EQ(frame_sink_source_map_.size(), 0u); in SurfaceManager's destructor,
when I close down the window. So I think an explicit call to unregister is still
needed in compositorFrameSink's own destructor, until we figure out why the
OnConnectionLost itself is not sufficient. I will put a TODO inside
OnCompositorFrameSinkConnectionLost.

+}
+
 }  // namespace content