Fix ReadPixels implementation specific read format/type on desktop GL.

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 5048 matching lines @@
         // Therefore if an error occurs we swallow the error and use the
         // internal implementation.
         if (params) {
           if (context_->HasExtension("GL_OES_read_format")) {
             ScopedGLErrorSuppressor suppressor("GLES2DecoderImpl::GetHelper",
                                                GetErrorState());
             glGetIntegerv(pname, params);
             if (glGetError() == GL_NO_ERROR)
               return true;
           }
-          *params = GLES2Util::GetPreferredGLReadPixelsFormat(
+          *params = GLES2Util::GetGLReadPixelsImplementationFormat(
               GetBoundReadFrameBufferInternalFormat());
         }
         return true;
       case GL_IMPLEMENTATION_COLOR_READ_TYPE:
         *num_written = 1;
         if (params) {
           if (context_->HasExtension("GL_OES_read_format")) {
             ScopedGLErrorSuppressor suppressor("GLES2DecoderImpl::GetHelper",
                                                GetErrorState());
             glGetIntegerv(pname, params);
             if (glGetError() == GL_NO_ERROR)
               return true;
           }
-          *params = GLES2Util::GetPreferredGLReadPixelsType(
+          *params = GLES2Util::GetGLReadPixelsImplementationType(
               GetBoundReadFrameBufferInternalFormat(),
               GetBoundReadFrameBufferTextureType());
         }
         return true;
       case GL_MAX_FRAGMENT_UNIFORM_VECTORS:
         *num_written = 1;
         if (params) {
           *params = group_->max_fragment_uniform_vectors();
         }
         return true;
@@ skipping 3651 matching lines @@
 
   GLenum src_internal_format = GetBoundReadFrameBufferInternalFormat();
   if (src_internal_format == 0) {
     LOCAL_SET_GL_ERROR(GL_INVALID_VALUE, "glReadPixels",
         "no valid read buffer source");
     return error::kNoError;
   }
   std::vector<GLenum> accepted_formats;
   std::vector<GLenum> accepted_types;
   switch (src_internal_format) {
-    case GL_RGB10_A2UI:
-      accepted_formats.push_back(GL_RGBA);
-      accepted_types.push_back(GL_UNSIGNED_INT_2_10_10_10_REV);
     case GL_R8UI:
     case GL_R16UI:
     case GL_R32UI:
     case GL_RG8UI:
     case GL_RG16UI:
     case GL_RG32UI:
     // All the RGB_INTEGER formats are not renderable.
     case GL_RGBA8UI:
+    case GL_RGB10_A2UI:
     case GL_RGBA16UI:
     case GL_RGBA32UI:
       accepted_formats.push_back(GL_RGBA_INTEGER);
       accepted_types.push_back(GL_UNSIGNED_INT);
       break;
     case GL_R8I:
     case GL_R16I:
     case GL_R32I:
     case GL_RG8I:
     case GL_RG16I:
     case GL_RG32I:
     case GL_RGBA8I:
     case GL_RGBA16I:
     case GL_RGBA32I:
       accepted_formats.push_back(GL_RGBA_INTEGER);
       accepted_types.push_back(GL_INT);
       break;
+    case GL_RGB10_A2:
+      accepted_formats.push_back(GL_RGBA);
+      accepted_types.push_back(GL_UNSIGNED_INT_2_10_10_10_REV);

[bajones, 2015/08/26 23:39:48]

Don't we need a break right here?

[Zhenyao Mo, 2015/08/26 23:43:20]

No, RGB10_A2 is special.  It will supports three format/type.  So I put it here
on purpose without a break.

[Zhenyao Mo, 2015/08/26 23:48:20]

Good point.  I will explicitly push the RGBA/UNSIGNED_BYTE here and break to
avoid any future maintenance cost. 

     default:
       accepted_formats.push_back(GL_RGBA);
       {
         GLenum src_type = GetBoundReadFrameBufferTextureType();
         switch (src_type) {
           case GL_HALF_FLOAT:
           case GL_HALF_FLOAT_OES:
           case GL_FLOAT:
           case GL_UNSIGNED_INT_10F_11F_11F_REV:
             accepted_types.push_back(GL_FLOAT);
@@ skipping 6261 matching lines @@
   return error::kNoError;
 }
 
 // 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