GPU: Keep track of the last real context and real surface made current.

ui/gl/gl_context.cc

Index: ui/gl/gl_context.cc
diff --git a/ui/gl/gl_context.cc b/ui/gl/gl_context.cc
index 496ed964e5e0f4f491556d4fe97f20169c8b32af..9229bb34c8cc39475d8492dbb8967cf9d8dcb4ed 100644
--- a/ui/gl/gl_context.cc
+++ b/ui/gl/gl_context.cc
@@ -20,6 +20,9 @@ namespace gfx {
 namespace {
 base::LazyInstance<base::ThreadLocalPointer<GLContext> >::Leaky
     current_context_ = LAZY_INSTANCE_INITIALIZER;
+
+base::LazyInstance<base::ThreadLocalPointer<GLContext> >::Leaky
+    current_real_context_ = LAZY_INSTANCE_INITIALIZER;
 }  // namespace
 
 GLContext::GLContext(GLShareGroup* share_group) : share_group_(share_group) {
@@ -90,7 +93,16 @@ GLContext* GLContext::GetCurrent() {
   return current_context_.Pointer()->Get();
 }
 
+GLContext* GLContext::GetRealCurrent() {
+  return current_real_context_.Pointer()->Get();
+}
+
 void GLContext::SetCurrent(GLContext* context, GLSurface* surface) {
+  DCHECK_EQ(!context, !surface);
+  if (!context || !context->IsVirtualContext()) {

[greggman, 2013/05/31 21:13:38]

I feel like the suggestion of GLRealContext vs GLVirtualContext vs GLContext
(the abstract class) would solve this without the need for IsVirtualContext

GLVirtualContext would call SetCurrent which would only set current_context_. 

It would also call shared_context_->SetAsRealContext()

GLRealContext would call SetCurrent and call

   GLContext::SetRealContext(this);

I don't know if I'm making any sense but you shouldn't have to know if this is
virtual. The virtual should delegate to its parent which might be real or
virtual. If virtual it would delegate to it's parent. etc.. Eventually you reach
a real context which would set current_real_context_

No need for "IsVirtualContext". No one should need to know that.

thoughts?

[jonathan.backer, 2013/06/03 14:57:23]

Done with some slight modifications. Mostly used GLContextReal (vs
GLRealContext) to follow the naming GLContextVirtual.

+    current_real_context_.Pointer()->Set(context);
+    GLSurface::SetRealCurrent(surface);
+  }
   current_context_.Pointer()->Set(context);
   GLSurface::SetCurrent(surface);
 }
@@ -128,7 +140,10 @@ void GLContext::SetupForVirtualization() {
 bool GLContext::MakeVirtuallyCurrent(
     GLContext* virtual_context, GLSurface* surface) {
   DCHECK(virtual_gl_api_);
-  return virtual_gl_api_->MakeCurrent(virtual_context, surface);
+  bool result = virtual_gl_api_->MakeCurrent(virtual_context, surface);
+  if (result)
+    SetCurrent(virtual_context, surface);
+  return result;
 }
 
 void GLContext::OnReleaseVirtuallyCurrent(GLContext* virtual_context) {
@@ -140,4 +155,8 @@ void GLContext::SetRealGLApi() {
   SetGLToRealGLApi();
 }
 
+bool GLContext::IsVirtualContext() {
+  return false;
+}
+
 }  // namespace gfx