ui::ContextFactory should not create SurfaceIdAllocators

content/browser/compositor/gpu_process_transport_factory.cc

Index: content/browser/compositor/gpu_process_transport_factory.cc
diff --git a/content/browser/compositor/gpu_process_transport_factory.cc b/content/browser/compositor/gpu_process_transport_factory.cc
index 821f642ac2a9ee5cfa4d12fa336c1345a82859fe..dab470e9c5b72a88d69ba1be4b47f5514140cffe 100644
--- a/content/browser/compositor/gpu_process_transport_factory.cc
+++ b/content/browser/compositor/gpu_process_transport_factory.cc
@@ -687,13 +687,8 @@ ui::ContextFactory* GpuProcessTransportFactory::GetContextFactory() {
   return this;
 }
 
-std::unique_ptr<cc::SurfaceIdAllocator>
-GpuProcessTransportFactory::CreateSurfaceIdAllocator() {
-  std::unique_ptr<cc::SurfaceIdAllocator> allocator =
-      base::WrapUnique(new cc::SurfaceIdAllocator(next_surface_client_id_++));
-  if (GetSurfaceManager())
-    allocator->RegisterSurfaceClientId(GetSurfaceManager());

[piman, 2016/07/21 17:07:11]

What does this now?

[Fady Samuel, 2016/07/21 17:15:01]

Ooops fixed. For context on where I'm going with this see this experimental
patch: https://codereview.chromium.org/2144733005/

I'd like SurfaceFactory to have a 1-1 correspondence to a single client and for
the local ID and nonce part of a surface ID to be allocated in the client.

-  return allocator;
+uint32_t GpuProcessTransportFactory::AllocateSurfaceClientId() {
+  return next_surface_client_id_++;
 }
 
 void GpuProcessTransportFactory::ResizeDisplay(ui::Compositor* compositor,