Ensure real GL Api is set before making GL calls.

ui/gl/gl_context_cgl.cc

Index: ui/gl/gl_context_cgl.cc
diff --git a/ui/gl/gl_context_cgl.cc b/ui/gl/gl_context_cgl.cc
index d826d81dc07f0e6d3304da4f1b2a849f96b0ffa7..0b63b4b3ea8485ce521ab8ed6fc6d2af1bd1c112 100644
--- a/ui/gl/gl_context_cgl.cc
+++ b/ui/gl/gl_context_cgl.cc
@@ -186,6 +186,9 @@ bool GLContextCGL::MakeCurrent(GLSurface* surface) {
     return false;
   }
 
+  // Set this as soon as the context is current, since we might call into GL.
+  SetRealGLApi();

[greggman, 2013/06/12 22:17:18]

Do you want to restore it if it fails?

I know that's a lot of refactoring but it seems like if it fails at line 193 or
like 198 then the previous RealGLApi should still be restored

Either that or it should be set to NULL so that if anyone is not checking the
results from MakeCurrent they'll fail rather than trashing someone else's
context.

What do you think?

+
   SetCurrent(surface);
   if (!InitializeExtensionBindings()) {
     ReleaseCurrent(surface);
@@ -197,7 +200,6 @@ bool GLContextCGL::MakeCurrent(GLSurface* surface) {
     return false;
   }
 
-  SetRealGLApi();
   return true;
 }