cc: Make OutputSurface::BindToClient pure virtual and not return bool

content/test/layouttest_support.cc

 // Copyright (c) 2012 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 "content/public/test/layouttest_support.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 369 matching lines @@
     attributes.alpha_size = -1;
     attributes.depth_size = 0;
     attributes.stencil_size = 0;
     attributes.samples = 0;
     attributes.sample_buffers = 0;
     attributes.bind_generates_resource = false;
     attributes.lose_context_when_out_of_memory = true;
     const bool automatic_flushes = false;
     const bool support_locking = false;
 
+    auto context_provider = make_scoped_refptr(new ContextProviderCommandBuffer(
+        gpu_channel_, gpu::GPU_STREAM_DEFAULT, gpu::GpuStreamPriority::NORMAL,
+        gpu::kNullSurfaceHandle,
+        GURL("chrome://gpu/"
+             "LayoutTestDependenciesImpl::CreateOutputSurface"),
+        automatic_flushes, support_locking, gpu::SharedMemoryLimits(),
+        attributes, nullptr,
+        command_buffer_metrics::OFFSCREEN_CONTEXT_FOR_TESTING));
+    context_provider->BindToCurrentThread();
+
     bool flipped_output_surface = false;
     return base::MakeUnique<cc::PixelTestOutputSurface>(
-        make_scoped_refptr(new ContextProviderCommandBuffer(
-            gpu_channel_, gpu::GPU_STREAM_DEFAULT,
-            gpu::GpuStreamPriority::NORMAL, gpu::kNullSurfaceHandle,
-            GURL("chrome://gpu/"
-                 "LayoutTestDependenciesImpl::CreateOutputSurface"),
-            automatic_flushes, support_locking, gpu::SharedMemoryLimits(),
-            attributes, nullptr,
-            command_buffer_metrics::OFFSCREEN_CONTEXT_FOR_TESTING)),
-        flipped_output_surface);
+        std::move(context_provider), flipped_output_surface);
   }
   void DisplayReceivedCompositorFrame(
       const cc::CompositorFrame& frame) override {}
   void DisplayWillDrawAndSwap(
       bool will_draw_and_swap,
       const cc::RenderPassList& render_passes) override {}
   void DisplayDidDrawAndSwap() override {}
 
  private:
   cc::TestCompositorFrameSink* FindCompositorFrameSink(int32_t routing_id) {
@@ skipping 710 matching lines @@
   return result;
 }
 
 void SchedulerRunIdleTasks(const base::Closure& callback) {
   blink::scheduler::RendererScheduler* scheduler =
       content::RenderThreadImpl::current()->GetRendererScheduler();
   blink::scheduler::RunIdleTasksForTesting(scheduler, callback);
 }
 
 }  // namespace content