Moved the logic of maintaining the current context to gles2 helper library.

webkit/plugins/ppapi/ppb_graphics_3d_impl.cc

 // Copyright (c) 2010 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 "webkit/plugins/ppapi/ppb_graphics_3d_impl.h"
 
 #include "gpu/command_buffer/common/command_buffer.h"
-#include "base/lazy_instance.h"
-#include "base/thread_local.h"
 #include "ppapi/c/dev/ppb_graphics_3d_dev.h"
 #include "webkit/plugins/ppapi/common.h"
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 namespace webkit {
 namespace ppapi {
 
 namespace {
 
-static base::LazyInstance<base::ThreadLocalPointer<PPB_Graphics3D_Impl> >
-    g_current_context_key(base::LINKER_INITIALIZED);
-
 // Size of the transfer buffer.
 enum { kTransferBufferSize = 512 * 1024 };
 
 PP_Bool IsGraphics3D(PP_Resource resource) {
   return BoolToPPBool(!!Resource::GetAs<PPB_Graphics3D_Impl>(resource));
 }
 
 PP_Bool GetConfigs(int32_t* configs, int32_t config_size, int32_t* num_config) {
   // TODO(neb): Implement me!
   return PP_FALSE;
@@ skipping 42 matching lines @@
   }
 
   return context->GetReference();
 }
 
 void* GetProcAddress(const char* name) {
   // TODO(neb): Implement me!
   return NULL;
 }
 
-PP_Bool MakeCurrent(PP_Resource graphics3d) {
-  if (!graphics3d) {
-    PPB_Graphics3D_Impl::ResetCurrent();
-    return PP_TRUE;
-  } else {
-    scoped_refptr<PPB_Graphics3D_Impl> context(
-        Resource::GetAs<PPB_Graphics3D_Impl>(graphics3d));
-    return BoolToPPBool(context.get() && context->MakeCurrent());
-  }
-}
-
-PP_Resource GetCurrentContext() {
-  PPB_Graphics3D_Impl* current_context = PPB_Graphics3D_Impl::GetCurrent();
-  return current_context ? current_context->GetReference() : 0;
-}
-
 PP_Bool SwapBuffers(PP_Resource graphics3d) {
   scoped_refptr<PPB_Graphics3D_Impl> context(
       Resource::GetAs<PPB_Graphics3D_Impl>(graphics3d));
   return BoolToPPBool(context && context->SwapBuffers());
 }
 
 uint32_t GetError() {
-  // Technically, this should return the last error that occurred on the current
-  // thread, rather than an error associated with a particular context.
-  // TODO(apatrick): Fix this.
-  PPB_Graphics3D_Impl* current_context = PPB_Graphics3D_Impl::GetCurrent();
-  if (!current_context)
-    return 0;
-
-  return current_context->GetError();
+  // TODO(alokp): Fix this.
+  return 0;
 }
 
 const PPB_Graphics3D_Dev ppb_graphics3d = {
   &IsGraphics3D,
   &GetConfigs,
   &ChooseConfig,
   &GetConfigAttrib,
   &QueryString,
   &CreateContext,
   &GetProcAddress,
-  &MakeCurrent,
-  &GetCurrentContext,
   &SwapBuffers,
   &GetError
 };
 
 }  // namespace
 
 PPB_Graphics3D_Impl::PPB_Graphics3D_Impl(PluginModule* module)
   : Resource(module),
     bound_instance_(NULL) {
 }
 
 const PPB_Graphics3D_Dev* PPB_Graphics3D_Impl::GetInterface() {
   return &ppb_graphics3d;
 }
 
-PPB_Graphics3D_Impl* PPB_Graphics3D_Impl::GetCurrent() {
-  return g_current_context_key.Get().Get();
-}
-
-void PPB_Graphics3D_Impl::ResetCurrent() {
-  g_current_context_key.Get().Set(NULL);
-}
-
 PPB_Graphics3D_Impl::~PPB_Graphics3D_Impl() {
   Destroy();
 }
 
 PPB_Graphics3D_Impl* PPB_Graphics3D_Impl::AsPPB_Graphics3D_Impl() {
   return this;
 }
 
 bool PPB_Graphics3D_Impl::Init(PP_Instance instance_id, int32_t config,
                                const int32_t* attrib_list) {
@@ skipping 36 matching lines @@
     // since the SwapBuffers is written to the command buffer and that command
     // buffer might be written to by another thread.
     // TODO(apatrick): Figure out the semantics of binding and resizing.
     platform_context_->SwapBuffers();
   }
 
   bound_instance_ = new_instance;
   return true;
 }
 
-bool PPB_Graphics3D_Impl::MakeCurrent() {
-  if (!platform_context_.get())
-    return false;
-
-  g_current_context_key.Get().Set(this);
-
-  // TODO(apatrick): Return false on context lost.
-  return true;
-}
-
 bool PPB_Graphics3D_Impl::SwapBuffers() {
   if (!platform_context_.get())
     return false;
 
   return platform_context_->SwapBuffers();
 }
 
 unsigned PPB_Graphics3D_Impl::GetError() {
   if (!platform_context_.get())
     return 0;
@@ skipping 16 matching lines @@
 }
 
 unsigned PPB_Graphics3D_Impl::GetBackingTextureId() {
   if (!platform_context_.get())
     return 0;
 
   return platform_context_->GetBackingTextureId();
 }
 
 void PPB_Graphics3D_Impl::Destroy() {
-  if (GetCurrent() == this) {
-    ResetCurrent();
-  }
-
   gles2_implementation_ = NULL;
-
   platform_context_.reset();
 }
 
 }  // namespace ppapi
 }  // namespace webkit