Call 'InitializeDynamicBindings()' before gl calls.

ui/gl/init/gl_factory_android.cc

 // Copyright 2016 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 "ui/gl/init/gl_factory.h"
 
 #include "base/logging.h"
 #include "base/trace_event/trace_event.h"
 #include "ui/gl/gl_bindings.h"
 #include "ui/gl/gl_context.h"
@@ skipping 42 matching lines @@
 GLNonOwnedContext::GLNonOwnedContext(GLShareGroup* share_group)
     : GLContextReal(share_group), display_(nullptr) {}
 
 bool GLNonOwnedContext::Initialize(GLSurface* compatible_surface,
                                    GpuPreference gpu_preference) {
   display_ = eglGetDisplay(EGL_DEFAULT_DISPLAY);
   return true;
 }
 
 bool GLNonOwnedContext::MakeCurrent(GLSurface* surface) {
   SetCurrent(surface);

[kylechar, 2016/08/25 13:37:22]

Unrelated to your change but it looks like all of the other
GLContext::MakeCurrent() overrides do the SetCurrent/SetRealGLAPi part in the
opposite order:

SetRealGLApi();
SetCurrent(surface);
InitializeDynamicBindings();

Not sure if that is important here? It looks like SetCurrent() can call
SetGLApi() so the order might change things.

   SetRealGLApi();
+  InitializeDynamicBindings();
   return true;
 }
 
 std::string GLNonOwnedContext::GetExtensions() {
   const char* extensions = eglQueryString(display_, EGL_EXTENSIONS);
   if (!extensions)
     return GLContext::GetExtensions();
 
   return GLContext::GetExtensions() + " " + extensions;
 }
@@ skipping 75 matching lines @@
     case kGLImplementationMockGL:
       return new GLSurfaceStub;
     default:
       NOTREACHED();
       return nullptr;
   }
 }
 
 }  // namespace init
 }  // namespace gl