Allow GL_TEXTURE_RECTANGLE_ARB to be the destination target for copyTextureCHROMIUM.

gpu/command_buffer/service/texture_manager.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/texture_manager.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 1911 matching lines @@
   ErrorState* error_state = state->GetErrorState();
   const Validators* validators = feature_info_->validators();
   if (((args.command_type == DoTexImageArguments::kTexImage2D) &&
        !validators->texture_target.IsValid(args.target)) ||
       ((args.command_type == DoTexImageArguments::kTexImage3D) &&
        !validators->texture_3_d_target.IsValid(args.target))) {
     ERRORSTATE_SET_GL_ERROR_INVALID_ENUM(
         error_state, function_name, args.target, "target");
     return false;
   }
-  // TODO(ccameron): Add a separate texture from |texture_target| for
-  // [Compressed]Tex[Sub]Image2D and related functions.

[erikchen, 2015/12/30 22:08:00]

I removed this to get my unit test to work. It's not clear to me why this is
necessary and correct, even when reading the the CL where it was introduced:
https://codereview.chromium.org/1376633002/

Perhaps someone with more experience could comment?

-  // http://crbug.com/536854
-  if (args.target == GL_TEXTURE_RECTANGLE_ARB) {
-    ERRORSTATE_SET_GL_ERROR_INVALID_ENUM(
-        error_state, function_name, args.target, "target");
-    return false;
-  }
+
   if (!ValidateTextureParameters(
       error_state, function_name, args.format, args.type,
       args.internal_format, args.level)) {
     return false;
   }
   if (!ValidForTarget(args.target, args.level,
                       args.width, args.height, args.depth) ||
       args.border != 0) {
     ERRORSTATE_SET_GL_ERROR(
         error_state, GL_INVALID_VALUE, function_name,
@@ skipping 664 matching lines @@
       return GL_HALF_FLOAT_OES;
     case GL_BGRA8_EXT:
       return GL_UNSIGNED_BYTE;
     default:
       return GL_NONE;
   }
 }
 
 }  // namespace gles2
 }  // namespace gpu