Mark SharedImage lost if cannot MakeCurrent

SkiaOutputSurfaceOnGpuImpl::DestroySkImage may not be able
to make the GL context current (e.g. tear down race or driver failure).
In this case, we should not issue any GL calls. Most of these
SkImages have an associated SharedImage. Eventually, all will.
This notifies the SharedImage representation not to glDeleteTextures
when the context is lost.

Bug: 923473
Change-Id: I80f9777aa36278b021febc7b5492a0310deb0639
Reviewed-on: https://chromium-review.googlesource.com/c/1439622
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: Peng Huang <penghuang@chromium.org>
Auto-Submit: Jonathan Backer <backer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#627982}

components/viz/service/display_embedder/skia_output_surface_impl.cc

@@ -164,6 +164,11 @@ class SkiaOutputSurfaceImpl::PromiseTextureHelper {
   static void Done(void* texture_context) {
     DCHECK(texture_context);
     auto* helper = static_cast<PromiseTextureHelper*>(texture_context);
+    if (helper->shared_image_) {
+      DCHECK(helper->impl_on_gpu_);
+      if (helper->impl_on_gpu_->was_context_lost())
+        helper->shared_image_->OnContextLost();
+    }
     delete helper;
   }
 

components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.h

@@ -142,6 +142,8 @@ class SkiaOutputSurfaceImplOnGpu : public gpu::ImageTransportSurfaceDelegate {
   void DestroySkImages(std::vector<sk_sp<SkImage>>&& images,
                        uint64_t sync_fence_release);
 
+  bool was_context_lost() { return context_state_->context_lost(); }
+
  private:
 // gpu::ImageTransportSurfaceDelegate implementation:
 #if defined(OS_WIN)