Delete RenderPassSink

cc/layers/render_surface_impl.cc

Index: cc/layers/render_surface_impl.cc
diff --git a/cc/layers/render_surface_impl.cc b/cc/layers/render_surface_impl.cc
index 4b6355631c136132effbc5ef218dd955f3d3e009..64ee09e10ef67e23d84de8c680717ae515a1ab15 100644
--- a/cc/layers/render_surface_impl.cc
+++ b/cc/layers/render_surface_impl.cc
@@ -357,7 +357,7 @@ int RenderSurfaceImpl::GetRenderPassId() {
   return id();
 }
 
-void RenderSurfaceImpl::AppendRenderPasses(RenderPassSink* pass_sink) {
+void RenderSurfaceImpl::AppendRenderPass(RenderPassSink* pass_sink) {

[danakj, 2017/03/13 14:46:00]

Now that this is the case, this function doesn't use anything private to
RenderSurfaceImpl. We could delete this function (and the whole RenderPassSink
concept) and just do this based on the public members of RenderSurfaceImpl at
the callsite, if you like.

[danakj, 2017/03/13 14:47:51]

Or, if you like, this could just return a unique_ptr<RenderPass> to the caller
if you wanna keep this code here. But either way RenderPassSink can go away now.

   std::unique_ptr<RenderPass> pass = RenderPass::Create(layer_list_.size());
   gfx::Rect damage_rect = GetDamageRect();
   damage_rect.Intersect(content_rect());