Defer compositor context creation to the thread.

cc/output/renderer_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/output/delegating_renderer.h"
 #include "cc/output/gl_renderer.h"
 #include "cc/output/output_surface.h"
 #include "cc/test/fake_output_surface_client.h"
 #include "cc/test/fake_renderer_client.h"
 #include "cc/test/fake_resource_provider.h"
 #include "cc/test/test_context_provider.h"
 #include "cc/test/test_web_graphics_context_3d.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace cc {
 namespace {
 
 class TestOutputSurface : public OutputSurface {
  public:
-  explicit TestOutputSurface(scoped_refptr<ContextProvider> context_provider);
-  ~TestOutputSurface() override;
+  explicit TestOutputSurface(
+      std::unique_ptr<ContextProvider::Factory> compositor_context_factory)
+      : OutputSurface(std::move(compositor_context_factory)) {}
+  ~TestOutputSurface() override = default;
 
-  // OutputSurface implementation
-  void SwapBuffers(CompositorFrame* frame) override;
+  // OutputSurface implementation.
+  void SwapBuffers(CompositorFrame* frame) override {
+    client_->DidSwapBuffers();
+    client_->DidSwapBuffersComplete();
+  }
 };
 
-TestOutputSurface::TestOutputSurface(
-    scoped_refptr<ContextProvider> context_provider)
-    : OutputSurface(std::move(context_provider)) {}
-
-TestOutputSurface::~TestOutputSurface() {
-}
-
-void TestOutputSurface::SwapBuffers(CompositorFrame* frame) {
-  client_->DidSwapBuffers();
-  client_->DidSwapBuffersComplete();
-}
-
 class MockContextProvider : public TestContextProvider {
  public:
-  explicit MockContextProvider(
-      std::unique_ptr<TestWebGraphicsContext3D> context)
-      : TestContextProvider(std::move(context)) {}
+  MockContextProvider()
+      : TestContextProvider(TestWebGraphicsContext3D::Create()) {}
+
+  class Factory : public ContextProvider::Factory {
+   public:
+    explicit Factory(scoped_refptr<MockContextProvider> provider)
+        : provider_(provider) {}
+
+    scoped_refptr<ContextProvider> CreateContext() override {
+      return std::move(provider_);
+    }
+
+   private:
+    scoped_refptr<MockContextProvider> provider_;
+  };
+
   MOCK_METHOD0(DeleteCachedResources, void());
 
  protected:
-  ~MockContextProvider() {}
+  ~MockContextProvider() = default;
 };
 
 template <class T>
 std::unique_ptr<Renderer> CreateRenderer(RendererClient* client,
                                          const RendererSettings* settings,
                                          OutputSurface* output_surface,
                                          ResourceProvider* resource_provider);
 
 template <>
 std::unique_ptr<Renderer> CreateRenderer<DelegatingRenderer>(
@@ skipping 12 matching lines @@
     OutputSurface* output_surface,
     ResourceProvider* resource_provider) {
   return GLRenderer::Create(
       client, settings, output_surface, resource_provider, NULL, 0);
 }
 
 template <typename T>
 class RendererTest : public ::testing::Test {
  protected:
   virtual void SetUp() {
-    context_provider_ =
-        new MockContextProvider(TestWebGraphicsContext3D::Create());
-    output_surface_.reset(new TestOutputSurface(context_provider_));
+    context_provider_ = new MockContextProvider;
+    output_surface_.reset(new TestOutputSurface(
+        base::MakeUnique<MockContextProvider::Factory>(context_provider_)));
     output_surface_->BindToClient(&output_surface_client_);
     resource_provider_ =
         FakeResourceProvider::Create(output_surface_.get(), nullptr);
     renderer_ = CreateRenderer<T>(&renderer_client_,
                                   &tree_settings_,
                                   output_surface_.get(),
                                   resource_provider_.get());
   }
 
   FakeRendererClient renderer_client_;
   RendererSettings tree_settings_;
   FakeOutputSurfaceClient output_surface_client_;
   scoped_refptr<MockContextProvider> context_provider_;
   std::unique_ptr<OutputSurface> output_surface_;
   std::unique_ptr<ResourceProvider> resource_provider_;
   std::unique_ptr<Renderer> renderer_;
 };
 
 typedef ::testing::Types<DelegatingRenderer, GLRenderer> RendererTypes;
 TYPED_TEST_CASE(RendererTest, RendererTypes);
 
 TYPED_TEST(RendererTest, ContextPurgedWhenRendererBecomesInvisible) {
-  EXPECT_CALL(*(this->context_provider_.get()), DeleteCachedResources())
-      .Times(1);
+  EXPECT_CALL(*this->context_provider_, DeleteCachedResources()).Times(1);
 
   EXPECT_TRUE(this->renderer_->visible());
   this->renderer_->SetVisible(false);
   EXPECT_FALSE(this->renderer_->visible());
 }
 
 }  // namespace
 }  // namespace cc