Fix disabled extensions for virtual contexts and respect command line.

ui/gl/gl_gl_api_implementation.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 "ui/gl/gl_gl_api_implementation.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 509 matching lines @@
 
 VirtualGLApi::~VirtualGLApi() {
 }
 
 void VirtualGLApi::Initialize(DriverGL* driver, GLContext* real_context) {
   InitializeBase(driver);
   real_context_ = real_context;
 
   DCHECK(real_context->IsCurrent(NULL));
   extensions_ = real_context->GetExtensions();
+  extensions_vec_ = base::SplitString(extensions_, " ", base::TRIM_WHITESPACE,
+                                      base::SPLIT_WANT_ALL);
 }
 
 bool VirtualGLApi::MakeCurrent(GLContext* virtual_context, GLSurface* surface) {
   bool switched_contexts = g_current_gl_context_tls->Get() != this;
   GLSurface* current_surface = GLSurface::GetCurrent();
   if (switched_contexts || surface != current_surface) {
     // MakeCurrent 'lite' path that avoids potentially expensive MakeCurrent()
     // calls if the GLSurface uses the same underlying surface or renders to
     // an FBO.
     if (switched_contexts || !current_surface ||
@@ skipping 53 matching lines @@
     return false;
   }
   return true;
 }
 
 void VirtualGLApi::OnReleaseVirtuallyCurrent(GLContext* virtual_context) {
   if (current_context_ == virtual_context)
     current_context_ = NULL;
 }
 
+void VirtualGLApi::glGetIntegervFn(GLenum pname, GLint* params) {
+  switch (pname) {
+    case GL_EXTENSIONS:

[no sievers, 2015/12/04 20:16:14]

did you mean GL_NUM_EXTENSIONS?

[David Yen, 2015/12/04 22:21:41]

Ah yes, thank you for catching this. Done.

+      *params = static_cast<GLint>(extensions_vec_.size());
+      break;
+    default:
+      driver_->fn.glGetIntegervFn(pname, params);
+      break;
+  }
+}
+
 const GLubyte* VirtualGLApi::glGetStringFn(GLenum name) {
   switch (name) {
     case GL_EXTENSIONS:
       return reinterpret_cast<const GLubyte*>(extensions_.c_str());
     default:
       return driver_->fn.glGetStringFn(name);
   }
 }
 
+const GLubyte* VirtualGLApi::glGetStringiFn(GLenum name, GLuint index) {
+  switch (name) {
+    case GL_EXTENSIONS:
+      if (index >= extensions_vec_.size())
+        return NULL;
+      return reinterpret_cast<const GLubyte*>(extensions_vec_[index].c_str());
+    default:
+      return driver_->fn.glGetStringiFn(name, index);
+  }
+}
+
 void VirtualGLApi::glFlushFn() {
   GLApiBase::glFlushFn();
 }
 
 void VirtualGLApi::glFinishFn() {
   GLApiBase::glFinishFn();
 }
 
 }  // namespace gfx