ui::ContextFactory should not create SurfaceIdAllocators

cc/surfaces/display_unittest.cc

Index: cc/surfaces/display_unittest.cc
diff --git a/cc/surfaces/display_unittest.cc b/cc/surfaces/display_unittest.cc
index b17a63a03effb11b1ab63772c5372042c5594f19..74c650802d2111bf5455bbefa9bff3b5f2c95cbc 100644
--- a/cc/surfaces/display_unittest.cc
+++ b/cc/surfaces/display_unittest.cc
@@ -99,7 +99,11 @@ class DisplayTest : public testing::Test {
       : factory_(&manager_, &surface_factory_client_),
         id_allocator_(kArbitraryClientId),
         task_runner_(new base::NullTaskRunner) {
-    id_allocator_.RegisterSurfaceClientId(&manager_);
+    manager_.RegisterSurfaceClientId(id_allocator_.client_id());
+  }
+
+  ~DisplayTest() override {
+    manager_.InvalidateSurfaceClientId(id_allocator_.client_id());
   }
 
   void SetUpDisplay(const RendererSettings& settings,