Use MojoCompositorFrameSink in RendererCompositorFrameSink

content/browser/renderer_host/render_widget_host_view_android.cc

Index: content/browser/renderer_host/render_widget_host_view_android.cc
diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc
index 7b57471e74b66a0f72b4dd2c10f5eb6ed5442955..6cc201d8d550aa31e28c6504dc7486b20b8acc1a 100644
--- a/content/browser/renderer_host/render_widget_host_view_android.cc
+++ b/content/browser/renderer_host/render_widget_host_view_android.cc
@@ -1111,8 +1111,10 @@ RenderWidgetHostViewAndroid::CreateSyntheticGestureTarget() {
 void RenderWidgetHostViewAndroid::SendReclaimCompositorResources(
     bool is_swap_ack) {
   DCHECK(host_);
-  host_->SendReclaimCompositorResources(is_swap_ack,
-                                        surface_returned_resources_);
+  renderer_compositor_frame_sink_->ReclaimResources(
+      surface_returned_resources_);
+  if (is_swap_ack)
+    renderer_compositor_frame_sink_->DidReceiveCompositorFrameAck();

[piman, 2017/04/10 23:35:58]

ditto: can we coalesce the IPCs?

[Saman Sami, 2017/04/11 00:09:52]

I think things can get better if I we send the IPCs in DelegatedFrameHostAndroid
instead of here. I'll try.

[Saman Sami, 2017/04/11 16:13:45]

Moving to DelegatedFrameHostAndroid doesn't help. Let's talk on slack with Fady
about this.

   surface_returned_resources_.clear();
 }
 
@@ -1130,8 +1132,10 @@ void RenderWidgetHostViewAndroid::ReclaimResources(
     SendReclaimCompositorResources(false /* is_swap_ack */);
 }
 
-void RenderWidgetHostViewAndroid::DidCreateNewRendererCompositorFrameSink() {
+void RenderWidgetHostViewAndroid::DidCreateNewRendererCompositorFrameSink(
+    cc::mojom::MojoCompositorFrameSinkClient* renderer_compositor_frame_sink) {
   delegated_frame_host_->CompositorFrameSinkChanged();
+  renderer_compositor_frame_sink_ = renderer_compositor_frame_sink;
   // Accumulated resources belong to the old RendererCompositorFrameSink and
   // should not be returned.
   surface_returned_resources_.clear();