Fix thread checking logic in ContextProviderCommandBuffer

content/common/gpu/client/context_provider_command_buffer.cc

Index: content/common/gpu/client/context_provider_command_buffer.cc
diff --git a/content/common/gpu/client/context_provider_command_buffer.cc b/content/common/gpu/client/context_provider_command_buffer.cc
index ba1baa9be5fd45aee51f58b752b365475af40e95..881a138671b826900953ccbfc32e267b3d3a82a6 100644
--- a/content/common/gpu/client/context_provider_command_buffer.cc
+++ b/content/common/gpu/client/context_provider_command_buffer.cc
@@ -68,7 +68,7 @@ ContextProviderCommandBuffer::~ContextProviderCommandBuffer() {
 
 
 CommandBufferProxyImpl* ContextProviderCommandBuffer::GetCommandBufferProxy() {
-  return WebContext3D()->GetCommandBufferProxy();
+  return WebContext3DNoChecks()->GetCommandBufferProxy();
 }
 
 WebGraphicsContext3DCommandBufferImpl*
@@ -119,11 +119,11 @@ void ContextProviderCommandBuffer::DetachFromThread() {
 gpu::gles2::GLES2Interface* ContextProviderCommandBuffer::ContextGL() {
   DCHECK(lost_context_callback_proxy_);  // Is bound to thread.
 
-  return WebContext3DNoChecks()->GetImplementation();
+  return WebContext3D()->GetImplementation();
 }
 
 gpu::ContextSupport* ContextProviderCommandBuffer::ContextSupport() {
-  return WebContext3D()->GetContextSupport();
+  return WebContext3DNoChecks()->GetContextSupport();
 }
 
 class GrContext* ContextProviderCommandBuffer::GrContext() {
@@ -152,7 +152,7 @@ void ContextProviderCommandBuffer::InvalidateGrContext(uint32_t state) {
 }
 
 void ContextProviderCommandBuffer::SetupLock() {
-  WebContext3D()->GetCommandBufferProxy()->SetLock(&context_lock_);
+  WebContext3DNoChecks()->GetCommandBufferProxy()->SetLock(&context_lock_);

[piman, 2015/11/20 04:54:35]

I think this one should stay as WebContext3D(), because we need to setup the
lock on the thread it was bound to (otherwise unsafe things would happen).

 }
 
 base::Lock* ContextProviderCommandBuffer::GetLock() {