RendererCompositorFrameSink should handle local surface id allocation

cc/surfaces/surface_manager.cc

Index: cc/surfaces/surface_manager.cc
diff --git a/cc/surfaces/surface_manager.cc b/cc/surfaces/surface_manager.cc
index 753191d7460be547e751b117afe584ad3734c89b..1a710d4c1f793111ef2cabecd8d3e9d3963de865 100644
--- a/cc/surfaces/surface_manager.cc
+++ b/cc/surfaces/surface_manager.cc
@@ -122,7 +122,7 @@ std::unique_ptr<Surface> SurfaceManager::CreateSurface(
   std::unique_ptr<Surface> surface = std::move(*it);
   surfaces_to_destroy_.erase(it);
   surface->set_destroyed(false);
-  DCHECK_EQ(surface_factory.get(), surface->factory().get());
+  surface->set_factory(surface_factory);

[Fady Samuel, 2017/03/15 12:05:47]

This is subtle and needs a unit test. Parent refers to a child, child
CompositorFrameSinkSupport goes away, and a new one is created, and the child
submits a new CompositorFrame, that ought to exercise this. I'd recommend doing
this in a separate patch.

[Saman Sami, 2017/03/16 18:33:06]

Done.

   return surface;
 }