Request context sharing via content::ContextProviderCommandBuffer

gpu/command_buffer/client/gles2_implementation.cc

Index: gpu/command_buffer/client/gles2_implementation.cc
diff --git a/gpu/command_buffer/client/gles2_implementation.cc b/gpu/command_buffer/client/gles2_implementation.cc
index 49835b833cf7bab2a10b4bb9907c5643dec4d0b3..b0a0a848f6090c48b9a7d0633aa4592367b53d8f 100644
--- a/gpu/command_buffer/client/gles2_implementation.cc
+++ b/gpu/command_buffer/client/gles2_implementation.cc
@@ -111,7 +111,7 @@ GLES2Implementation::SingleThreadChecker::~SingleThreadChecker() {
 
 GLES2Implementation::GLES2Implementation(
     GLES2CmdHelper* helper,
-    ShareGroup* share_group,
+    scoped_refptr<ShareGroup> share_group,
     TransferBufferInterface* transfer_buffer,
     bool bind_generates_resource,
     bool lose_context_when_out_of_memory,
@@ -186,6 +186,7 @@ GLES2Implementation::GLES2Implementation(
                          bind_generates_resource,
                          gpu_control_->GetCommandBufferID().GetUnsafeValue()));
   DCHECK(share_group_->bind_generates_resource() == bind_generates_resource);
+  DCHECK(!share_group_->IsLost());

[piman, 2016/04/27 22:09:33]

I don't think we want this DCHECK - another thread may have lost the share group
after it's passed to the constructor. The GLES2Implementation maybe DOA, but
such is life?

[danakj, 2016/04/28 00:25:58]

Ah.. ya fair point. Removed and moved the check to IsLost in the context
provider to be *after* the GLES2Impl is set up.

 
   memset(&reserved_ids_, 0, sizeof(reserved_ids_));
 }