sRGB transition is invalid when context is ES.

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 <vector>
 
 #include "base/command_line.h"
 #include "base/stl_util.h"
@@ skipping 88 matching lines @@
         gl_internal_format = GL_RGBA;
         break;
       case GL_RGB:
         gl_internal_format = GL_RGB;
         break;
       default:
         break;
     }
   }
 
+  if (gfx::g_version_info->IsAtLeastGL(2, 1) ||
+      gfx::g_version_info->IsAtLeastGLES(3, 0)) {
+    switch (internal_format) {
+      case GL_SRGB_EXT:
+        gl_internal_format = GL_SRGB8;
+        break;
+      case GL_SRGB_ALPHA_EXT:
+        gl_internal_format = GL_SRGB8_ALPHA8;
+        break;
+      default:
+        break;
+    }
+  }
+
   if (gfx::g_version_info->is_es)

[xinghua.cao, 2016/05/10 11:02:30]

If driver context is es, it will return directly. So sRGB transition should be
before it, Or will report error.

     return gl_internal_format;
 
   if (type == GL_FLOAT) {
     switch (internal_format) {
       // We need to map all the unsized internal formats from ES2 clients.
       case GL_RGBA:
         gl_internal_format = GL_RGBA32F_ARB;
         break;
       case GL_RGB:
         gl_internal_format = GL_RGB32F_ARB;
@@ skipping 28 matching lines @@
         break;
       case GL_ALPHA:
         gl_internal_format = GL_ALPHA16F_ARB;
         break;
       default:
         NOTREACHED();
         break;
     }
   }
 
-  if (gfx::g_version_info->IsAtLeastGL(2, 1) ||
-      gfx::g_version_info->IsAtLeastGLES(3, 0)) {
-    switch (internal_format) {
-      case GL_SRGB_EXT:
-        gl_internal_format = GL_SRGB8;
-        break;
-      case GL_SRGB_ALPHA_EXT:
-        gl_internal_format = GL_SRGB8_ALPHA8;
-        break;
-      default:
-        break;
-    }
-  }
-
   return gl_internal_format;
 }
 
 static inline GLenum GetTexFormat(GLenum format) {
   GLenum gl_format = format;
 
   DCHECK(gfx::g_version_info);
   if (gfx::g_version_info->IsAtLeastGL(2, 1) ||
       gfx::g_version_info->IsAtLeastGLES(3, 0)) {
     switch (format) {
@@ skipping 484 matching lines @@
 
 void VirtualGLApi::glFlushFn() {
   GLApiBase::glFlushFn();
 }
 
 void VirtualGLApi::glFinishFn() {
   GLApiBase::glFinishFn();
 }
 
 }  // namespace gfx