Corrected test that blocked ES3 compressions formats with 2D textures.

gpu/command_buffer/service/gles2_cmd_decoder.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/gles2_cmd_decoder.h"
 
 #include <stdio.h>
 
 #include <algorithm>
 #include <cmath>
@@ skipping 8995 matching lines @@
     case GL_COMPRESSED_RGB8_PUNCHTHROUGH_ALPHA1_ETC2:
     case GL_COMPRESSED_SRGB8_PUNCHTHROUGH_ALPHA1_ETC2:
     case GL_COMPRESSED_RGBA8_ETC2_EAC:
     case GL_COMPRESSED_SRGB8_ALPHA8_ETC2_EAC:
       if (width <= 0 || height <= 0 || depth <= 0) {
         LOCAL_SET_GL_ERROR(
             GL_INVALID_OPERATION, function_name,
             "width, height, or depth invalid");
         return false;
       }
-      if (target != GL_TEXTURE_2D_ARRAY) {
+      if (target == GL_TEXTURE_3D) {
         LOCAL_SET_GL_ERROR(
             GL_INVALID_OPERATION, function_name,
             "target invalid for format");
         return false;
       }
       return true;
     default:
       return false;
   }
 }
@@ skipping 81 matching lines @@
     case GL_COMPRESSED_SIGNED_RG11_EAC:
     case GL_COMPRESSED_RGB8_ETC2:
     case GL_COMPRESSED_SRGB8_ETC2:
     case GL_COMPRESSED_RGB8_PUNCHTHROUGH_ALPHA1_ETC2:
     case GL_COMPRESSED_SRGB8_PUNCHTHROUGH_ALPHA1_ETC2:
     case GL_COMPRESSED_RGBA8_ETC2_EAC:
     case GL_COMPRESSED_SRGB8_ALPHA8_ETC2_EAC:
       {
         const int kBlockSize = 4;
         GLsizei tex_width, tex_height;
         if (target != GL_TEXTURE_2D_ARRAY ||

[Zhenyao Mo, 2015/05/08 18:16:15]

Also here.

             !texture->GetLevelSize(target, level,
                                    &tex_width, &tex_height, nullptr) ||
             (xoffset % kBlockSize) || (yoffset % kBlockSize) ||
             ((width % kBlockSize) && xoffset + width != tex_width) ||
             ((height % kBlockSize) && yoffset + height != tex_height)) {
           LOCAL_SET_GL_ERROR(
               GL_INVALID_OPERATION, function_name,
               "dimensions must match existing texture level dimensions");
           return false;
         }
@@ skipping 3983 matching lines @@
   }
 }
 
 // Include the auto-generated part of this file. We split this because it means
 // we can easily edit the non-auto generated parts right here in this file
 // instead of having to edit some template or the code generator.
 #include "gpu/command_buffer/service/gles2_cmd_decoder_autogen.h"
 
 }  // namespace gles2
 }  // namespace gpu