gpu: Remove WebGraphicsContext3D::makeContextCurrent()

content/renderer/render_thread_impl.cc

 // Copyright (c) 2012 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/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 1159 matching lines @@
     if (SynchronousCompositorFactory* factory =
             SynchronousCompositorFactory::GetInstance()) {
       shared_main_thread_contexts_ = factory->CreateOffscreenContextProvider(
           GetOffscreenAttribs(), "Offscreen-MainThread");
     }
 #endif
     if (!shared_main_thread_contexts_) {
       shared_main_thread_contexts_ = ContextProviderCommandBuffer::Create(
           CreateOffscreenContext3d(), "Offscreen-MainThread");
     }
+    if (shared_main_thread_contexts_ &&
+        !shared_main_thread_contexts_->BindToCurrentThread())

[dshwang, 2014/08/15 14:54:01]

BindToCurrentThread() should be called one time when creating.

This method is used by Accelerated 2D Canvas. Chromium returns the context after
binding already.

[danakj, 2014/08/15 15:28:44]

This seems like a separate change, can you do this change by itself?

[dshwang, 2014/08/15 16:02:32]

Ok, I'll separate it on another CL.

+      shared_main_thread_contexts_ = NULL;
   }
-  if (shared_main_thread_contexts_ &&
-      !shared_main_thread_contexts_->BindToCurrentThread())
-    shared_main_thread_contexts_ = NULL;
   return shared_main_thread_contexts_;
 }
 
 AudioRendererMixerManager* RenderThreadImpl::GetAudioRendererMixerManager() {
   if (!audio_renderer_mixer_manager_) {
     audio_renderer_mixer_manager_.reset(new AudioRendererMixerManager(
         GetAudioHardwareConfig()));
   }
 
   return audio_renderer_mixer_manager_.get();
@@ skipping 459 matching lines @@
   hidden_widget_count_--;
 
   if (!GetContentClient()->renderer()->RunIdleHandlerWhenWidgetsHidden()) {
     return;
   }
 
   ScheduleIdleHandler(kLongIdleHandlerDelayMs);
 }
 
 }  // namespace content