ui::ContextFactory should not create SurfaceIdAllocators

cc/test/test_delegating_output_surface.cc

Index: cc/test/test_delegating_output_surface.cc
diff --git a/cc/test/test_delegating_output_surface.cc b/cc/test/test_delegating_output_surface.cc
index 15fca6c14cf857cfc899f5435236b786cec5a60b..dbbf59207c4f2a6669edcd817dc4ec271f9ed2f6 100644
--- a/cc/test/test_delegating_output_surface.cc
+++ b/cc/test/test_delegating_output_surface.cc
@@ -56,10 +56,14 @@ TestDelegatingOutputSurface::TestDelegatingOutputSurface(
   capabilities_.delegated_sync_points_required =
       !context_shared_with_compositor;
 
-  surface_id_allocator_->RegisterSurfaceClientId(surface_manager_.get());
+  surface_manager_->RegisterSurfaceClientId(surface_id_allocator_->client_id());
 }
 
-TestDelegatingOutputSurface::~TestDelegatingOutputSurface() {}
+TestDelegatingOutputSurface::~TestDelegatingOutputSurface() {
+  if (surface_manager_)

[piman, 2016/07/21 21:02:22]

nit: {}

[Fady Samuel, 2016/07/21 21:21:03]

actually TestDelegatingOuptutSurface owns SurfaceManager, so I don't need this
test.

+    surface_manager_->InvalidateSurfaceClientId(
+        surface_id_allocator_->client_id());
+}
 
 bool TestDelegatingOutputSurface::BindToClient(OutputSurfaceClient* client) {
   if (!OutputSurface::BindToClient(client))