Bug 40398 – Support UNPACK_FLIP_Y_WEBGL and UNPACK_PREMULTIPLY_ALPHA_WEBGL for texImage2D taking ArrayBufferView

• Status: RESOLVED
• Resolution: FIXED
• Component: WebGL

[Kenneth Russell]

The new pixel store parameters UNPACK_FLIP_Y_WEBGL and UNPACK_PREMULTIPLY_ALPHA_WEBGL need to be supported for the texImage2D and texSubImage2D entry points taking ArrayBufferView.

This bug depends on 40318.

[Kenneth Russell]

Created attachment 59964 [details]
Patch

From the ChangeLog:

Added support for UNPACK_FLIP_Y_WEBGL and UNPACK_PREMULTIPLY_ALPHA_WEBGL pixel store parameters to texImage2D and texSubImage2D entry points taking ArrayBufferView. More cleanly separated the unpacking and packing phases of user-supplied pixel data in GraphicsContext3D, and added support for unpack alignment. Fixed bug in handling of unpackAlignment in GraphicsContext3D::flipVertically. Necessarily added validation of the amount of data passed to texImage2D and texSubImage2D. Modified fast/canvas/webgl/tex-image-with-format-and-type.html to include premultiplied alpha tests for relevant source formats; added new test which exercises all combinations of UNPACK_FLIP_Y_WEBGL, UNPACK_PREMULTIPLY_ALPHA_WEBGL, and UNPACK_ALIGNMENT pixel store parameters.

Note to reviewers:

This patch is large, but the individual code changes are small and easily reviewable, and it is not possible to break it down into any smaller set of patches which are individually testable. The test cases and their boilerplate output comprise more than half of the patch. The test cases are as exhaustive as possible and caught a couple of implementation errors.

[Eric Seidel (no email)]

Attachment 59964 [details] did not build on mac:
Build output: http://webkit-commit-queue.appspot.com/results/3268898

[Kenneth Russell]

Created attachment 59970 [details]
Revised patch

Revised patch fixing build problem in Release mode on Mac

[Dimitri Glazkov (Google)]

Comment on attachment 59970 [details]
Revised patch

r=me (pair programming FTW!), except:

WebCore/html/canvas/WebGLRenderingContext.cpp:3567
 +          return false;
Need to synthesize an error here.

[Kenneth Russell]

Committed r62145: <http://trac.webkit.org/changeset/62145>

[Kenneth Russell]

*** Bug 32619 has been marked as a duplicate of this bug. ***

[Kenneth Russell]

*** Bug 32040 has been marked as a duplicate of this bug. ***