Reduce SurfaceIdAllocator usage and tie SurfaceFactory to a single FrameSinkId

services/ui/ws/server_window_surface.cc

Index: services/ui/ws/server_window_surface.cc
diff --git a/services/ui/ws/server_window_surface.cc b/services/ui/ws/server_window_surface.cc
index 9ca1b9e1c84b22e4996f6e9a4c79a90e326f7736..9560b430b708a85e9172247341b1b2719af89009 100644
--- a/services/ui/ws/server_window_surface.cc
+++ b/services/ui/ws/server_window_surface.cc
@@ -21,17 +21,17 @@ ServerWindowSurface::ServerWindowSurface(
     ServerWindowSurfaceManager* manager,
     mojo::InterfaceRequest<Surface> request,
     mojom::SurfaceClientPtr client)
-    : manager_(manager),
-      surface_id_allocator_(cc::FrameSinkId(
+    : frame_sink_id_(
           manager->window()->delegate()->GetSurfacesState()->next_client_id(),
-          0)),
-      surface_factory_(manager_->GetSurfaceManager(), this),
+          0),
+      manager_(manager),
+      surface_id_allocator_(frame_sink_id_),
+      surface_factory_(frame_sink_id_, manager_->GetSurfaceManager(), this),
       client_(std::move(client)),
       binding_(this, std::move(request)) {
   cc::SurfaceManager* surface_manager = manager_->GetSurfaceManager();
-  surface_manager->RegisterFrameSinkId(surface_id_allocator_.frame_sink_id());
-  surface_manager->RegisterSurfaceFactoryClient(
-      surface_id_allocator_.frame_sink_id(), this);
+  surface_manager->RegisterFrameSinkId(frame_sink_id_);
+  surface_manager->RegisterSurfaceFactoryClient(frame_sink_id_, this);
 }
 
 ServerWindowSurface::~ServerWindowSurface() {
@@ -40,9 +40,8 @@ ServerWindowSurface::~ServerWindowSurface() {
   // |surface_factory_|'s resources early on.
   surface_factory_.DestroyAll();
   cc::SurfaceManager* surface_manager = manager_->GetSurfaceManager();
-  surface_manager->UnregisterSurfaceFactoryClient(
-      surface_id_allocator_.frame_sink_id());
-  surface_manager->InvalidateFrameSinkId(surface_id_allocator_.frame_sink_id());
+  surface_manager->UnregisterSurfaceFactoryClient(frame_sink_id_);
+  surface_manager->InvalidateFrameSinkId(frame_sink_id_);
 }
 
 void ServerWindowSurface::SubmitCompositorFrame(