gpu: Thread-safe command buffer state lookup.

content/common/gpu/client/context_provider_command_buffer.cc

 // Copyright (c) 2013 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/common/gpu/client/context_provider_command_buffer.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <set>
@@ skipping 274 matching lines @@
         attributes_.lose_context_when_out_of_memory, support_client_side_arrays,
         command_buffer_.get()));
     if (!gles2_impl_->Initialize(memory_limits_.start_transfer_buffer_size,
                                  memory_limits_.min_transfer_buffer_size,
                                  memory_limits_.max_transfer_buffer_size,
                                  memory_limits_.mapped_memory_reclaim_limit)) {
       DLOG(ERROR) << "Failed to initialize GLES2Implementation.";
       return false;
     }
 
-    if (command_buffer_->GetLastError() != gpu::error::kNoError) {
+    if (command_buffer_->GetLastState().error != gpu::error::kNoError) {
       DLOG(ERROR) << "Context dead on arrival. Last error: "
-                  << command_buffer_->GetLastError();
+                  << command_buffer_->GetLastState().error;
       return false;
     }
 
     // If any context in the share group has been lost, then abort and don't
     // continue since we need to go back to the caller of the constructor to
     // find the correct share group.
     // This may happen in between the share group being chosen at the
     // constructor, and getting to run this BindToCurrentThread method which
     // can be on some other thread.
     // We intentionally call this *after* creating the command buffer via the
@@ skipping 139 matching lines @@
     const base::trace_event::MemoryDumpArgs& args,
     base::trace_event::ProcessMemoryDump* pmd) {
   DCHECK(bind_succeeded_);
   if (!gr_context_)
     return false;
 
   base::Optional<base::AutoLock> hold;
   if (support_locking_)
     hold.emplace(context_lock_);
 
+  gles2_impl_->OnMemoryDump(args, pmd);

[ericrk, 2016/12/08 01:59:39]

Seems like this will fix crbug.com/638862, can you close that out when this
lands? Thanks!

+
   context_thread_checker_.DetachFromThread();
   SkiaGpuTraceMemoryDump trace_memory_dump(
       pmd, gles2_impl_->ShareGroupTracingGUID());
   gr_context_->get()->dumpMemoryStatistics(&trace_memory_dump);
   context_thread_checker_.DetachFromThread();
   return true;
 }
 
 }  // namespace content