SRGB_EXT is a valid format of texture in WebGL1.0 and ES2.0 contexts

gpu/command_buffer/service/feature_info.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 "gpu/command_buffer/service/feature_info.h"
 
 #include <stddef.h>
 
 #include <set>
 #include <vector>
@@ skipping 535 matching lines @@
        extensions.Contains("GL_EXT_sRGB")) ||
       gfx::HasDesktopGLFeatures()) {
     AddExtensionString("GL_EXT_sRGB");
     validators_.texture_internal_format.AddValue(GL_SRGB_EXT);
     validators_.texture_internal_format.AddValue(GL_SRGB_ALPHA_EXT);
     validators_.texture_format.AddValue(GL_SRGB_EXT);
     validators_.texture_format.AddValue(GL_SRGB_ALPHA_EXT);
     validators_.render_buffer_format.AddValue(GL_SRGB8_ALPHA8_EXT);
     validators_.frame_buffer_parameter.AddValue(
         GL_FRAMEBUFFER_ATTACHMENT_COLOR_ENCODING_EXT);
-    validators_.texture_unsized_internal_format.AddValue(GL_SRGB_ALPHA_EXT);
+    if (context_type_ == CONTEXT_TYPE_WEBGL1 ||
+        context_type_ == CONTEXT_TYPE_OPENGLES2) {

[Zhenyao Mo, 2016/04/27 17:12:26]

This should be part of the "if" condition in line 545. We don't need to expose
GL_EXT_sRGB otherwise.

[xinghua.cao, 2016/04/28 09:59:27]

Done.

+      validators_.texture_unsized_internal_format.AddValue(GL_SRGB_EXT);
+      validators_.texture_unsized_internal_format.AddValue(GL_SRGB_ALPHA_EXT);
+    }
   }
 
   bool enable_texture_format_bgra8888 = false;
   bool enable_read_format_bgra = false;
   bool enable_render_buffer_bgra = false;
   bool enable_immutable_texture_format_bgra_on_es3 =
       extensions.Contains("GL_APPLE_texture_format_BGRA8888");
 
   // Check if we should allow GL_EXT_texture_format_BGRA8888.
   if (extensions.Contains("GL_EXT_texture_format_BGRA8888") ||
@@ skipping 800 matching lines @@
   if (pos == std::string::npos) {
     extensions_ += (extensions_.empty() ? "" : " ") + str;
   }
 }
 
 FeatureInfo::~FeatureInfo() {
 }
 
 }  // namespace gles2
 }  // namespace gpu