[exo]CompositorFrameSinkHolder calls WillDraw in DidReceiveCompositorFrameAck

components/exo/compositor_frame_sink_holder.cc

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 c21346fc51b71d6979652fc38cf8e9ba1c5c742b..8eeefe09060c57069f361e229e2bfa7fe9ecf203 100644
--- a/components/exo/compositor_frame_sink_holder.cc
+++ b/components/exo/compositor_frame_sink_holder.cc
@@ -42,6 +42,8 @@ void CompositorFrameSinkHolder::SetResourceReleaseCallback(
 
 void CompositorFrameSinkHolder::DidReceiveCompositorFrameAck() {
   // TODO(staraz): Implement this

[reveman, 2017/04/06 20:42:04]

nit: remove this now?

[Alex Z., 2017/04/06 20:49:13]

Done.

+  if (surface_)
+    surface_->WillDraw();

[Fady Samuel, 2017/04/06 20:52:33]

Rename WillDraw to DidReceiveCompositorFrameAck?

[reveman, 2017/04/06 21:09:06]

+1

[Alex Z., 2017/04/06 21:11:12]

Done.

 }
 
 void CompositorFrameSinkHolder::OnBeginFrame(const cc::BeginFrameArgs& args) {
@@ -62,10 +64,7 @@ void CompositorFrameSinkHolder::ReclaimResources(
 
 void CompositorFrameSinkHolder::WillDrawSurface(
     const cc::LocalSurfaceId& local_surface_id,
-    const gfx::Rect& damage_rect) {
-  if (surface_)
-    surface_->WillDraw();
-}
+    const gfx::Rect& damage_rect) {}
 
 ////////////////////////////////////////////////////////////////////////////////
 // cc::ExternalBeginFrameSourceClient overrides: