sRGB transition is invalid when context is ES.

ui/gl/gl_gl_api_implementation.cc

Index: ui/gl/gl_gl_api_implementation.cc
diff --git a/ui/gl/gl_gl_api_implementation.cc b/ui/gl/gl_gl_api_implementation.cc
index c0fc5bba0dcb76bbbc928f4ecf0f11f554385674..96a5cd10269ec6815933e36d6510b25106189025 100644
--- a/ui/gl/gl_gl_api_implementation.cc
+++ b/ui/gl/gl_gl_api_implementation.cc
@@ -106,6 +106,20 @@ static inline GLenum GetTexInternalFormat(GLenum internal_format,
     }
   }
 
+  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;
 
@@ -155,20 +169,6 @@ static inline GLenum GetTexInternalFormat(GLenum internal_format,
     }
   }
 
-  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;
 }