Fix webkit layout tests with SkDDL

SendResults can't be called on GpuMain thread or we crash. This CL
calls it from display compositor thread after we've ensured that
the appropriate GL context is current.

Change-Id: Ifa5c2cffafb5cfa55e5b21b1b93b6f1bc8d6e1e1
Reviewed-on: https://chromium-review.googlesource.com/c/1351577
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: weiliangc <weiliangc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611395}

components/viz/service/display_embedder/skia_output_surface_impl.cc

@@ -556,6 +556,8 @@ void SkiaOutputSurfaceImpl::CopyOutput(
     const gfx::Rect& copy_rect,
     std::unique_ptr<CopyOutputRequest> request) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
+  if (!request->has_result_task_runner())
+    request->set_result_task_runner(base::ThreadTaskRunnerHandle::Get());
   auto sequence_id = gpu_service_->skia_output_surface_sequence_id();
   auto callback = base::BindOnce(&SkiaOutputSurfaceImplOnGpu::CopyOutput,
                                  base::Unretained(impl_on_gpu_.get()), id,

components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.cc

@@ -292,6 +292,11 @@ void SkiaOutputSurfaceImplOnGpu::CopyOutput(
     const gfx::Rect& copy_rect,
     std::unique_ptr<CopyOutputRequest> request) {
   DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
+  if (!MakeCurrent()) {
+    request->SendResult(
+        std::make_unique<CopyOutputSkBitmapResult>(gfx::Rect(), SkBitmap()));
+    return;
+  }
 
   // TODO(crbug.com/644851): Complete the implementation for all request types,
   // scaling, etc.