Add texture swizzle back to command buffer.

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 1473 matching lines @@
         kTotalDrawBufferEnums - max_draw_buffers);
   }
 
   if (feature_flags_.ext_texture_format_bgra8888) {
     validators_.texture_internal_format.AddValue(GL_BGRA8_EXT);
     validators_.texture_sized_color_renderable_internal_format.AddValue(
         GL_BGRA8_EXT);
     validators_.texture_sized_texture_filterable_internal_format.AddValue(
         GL_BGRA8_EXT);
   }
+
+  if (!IsWebGLContext()) {
+    validators_.texture_parameter.AddValue(GL_TEXTURE_SWIZZLE_R);
+    validators_.texture_parameter.AddValue(GL_TEXTURE_SWIZZLE_G);
+    validators_.texture_parameter.AddValue(GL_TEXTURE_SWIZZLE_B);
+    validators_.texture_parameter.AddValue(GL_TEXTURE_SWIZZLE_A);

[Ken Russell (switch to Gerrit), 2016/12/01 22:46:27]

What about an understanding of the valid values for these parameters?

[Zhenyao Mo, 2016/12/01 22:47:51]

It's already in place. I'll add a test for that.

+  }
 }
 
 bool FeatureInfo::IsWebGLContext() const {
   return IsWebGLContextType(context_type_);
 }
 
 bool FeatureInfo::IsWebGL1OrES2Context() const {
   // Switch statement to cause a compile-time error if we miss a case.
   switch (context_type_) {
     case CONTEXT_TYPE_WEBGL1:
@@ skipping 35 matching lines @@
   if (pos == std::string::npos) {
     extensions_ += (extensions_.empty() ? "" : " ") + str;
   }
 }
 
 FeatureInfo::~FeatureInfo() {
 }
 
 }  // namespace gles2
 }  // namespace gpu