Handle pixel (un)pack buffers with non-zero offset.

gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc

 // Copyright (c) 2016 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_passthrough.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "ui/gl/gl_version_info.h"
 
 namespace gpu {
@@ skipping 449 matching lines @@
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoCompressedTexImage2D(
     GLenum target,
     GLint level,
     GLenum internalformat,
     GLsizei width,
     GLsizei height,
     GLint border,
     GLsizei imageSize,
+    GLsizei dataSize,

[Zhenyao Mo, 2017/04/11 19:56:54]

nit: this is actually against chromium coding style.

imageSize -> image_size, dataSize -> data_size

[Corentin Wallez, 2017/04/11 22:06:34]

Done here and in other places in this file and the header.

     const void* data) {
+  // TODO(cwallez@chromium.org): Use dataSize with the robust version of the
+  // entry point
   glCompressedTexImage2D(target, level, internalformat, width, height, border,
                          imageSize, data);
   return error::kNoError;
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoCompressedTexSubImage2D(
     GLenum target,
     GLint level,
     GLint xoffset,
     GLint yoffset,
     GLsizei width,
     GLsizei height,
     GLenum format,
     GLsizei imageSize,
+    GLsizei dataSize,
     const void* data) {
+  // TODO(cwallez@chromium.org): Use dataSize with the robust version of the
+  // entry point
   glCompressedTexSubImage2D(target, level, xoffset, yoffset, width, height,
                             format, imageSize, data);
   return error::kNoError;
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoCompressedTexImage3D(
     GLenum target,
     GLint level,
     GLenum internalformat,
     GLsizei width,
     GLsizei height,
     GLsizei depth,
     GLint border,
     GLsizei imageSize,
+    GLsizei dataSize,
     const void* data) {
+  // TODO(cwallez@chromium.org): Use dataSize with the robust version of the
+  // entry point
   glCompressedTexImage3D(target, level, internalformat, width, height, depth,
                          border, imageSize, data);
   return error::kNoError;
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoCompressedTexSubImage3D(
     GLenum target,
     GLint level,
     GLint xoffset,
     GLint yoffset,
     GLint zoffset,
     GLsizei width,
     GLsizei height,
     GLsizei depth,
     GLenum format,
     GLsizei imageSize,
+    GLsizei dataSize,
     const void* data) {
+  // TODO(cwallez@chromium.org): Use dataSize with the robust version of the
+  // entry point
   glCompressedTexSubImage3D(target, level, xoffset, yoffset, zoffset, width,
                             height, depth, format, imageSize, data);
   return error::kNoError;
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoCopyBufferSubData(
     GLenum readtarget,
     GLenum writetarget,
     GLintptr readoffset,
     GLintptr writeoffset,
@@ skipping 3441 matching lines @@
 }
 
 error::Error GLES2DecoderPassthroughImpl::DoSetEnableDCLayersCHROMIUM(
     GLboolean enable) {
   NOTIMPLEMENTED();
   return error::kNoError;
 }
 
 }  // namespace gles2
 }  // namespace gpu