Implement the new texStorage methods void texStorage2D(GLenum target, GLsizei levels, GLenum internalformat, GLsizei width, GLsizei height); void texStorage3D(GLenum target, GLsizei levels, GLenum internalformat, GLsizei width, GLsizei height, GLsizei depth); void texSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLenum type, ArrayBufferView? pixels); void texSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLenum type, ImageData? data); void texSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLenum type, HTMLImageElement image); void texSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLenum type, HTMLCanvasElement canvas); void texSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLenum type, HTMLVideoElement video); void copyTexSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLint x, GLint y, GLsizei width, GLsizei height); (Maybe 3d should be split into a separate bug)
<rdar://problem/15002343>
Also: void compressedTexImage3D(GLenum target, GLint level, GLenum internalformat, GLsizei width, GLsizei height, GLsizei depth, GLint border, GLsizei imageSize, ArrayBufferView data); void compressedTexSubImage3D(GLenum target, GLint level, GLint xoffset, GLint yoffset, GLint zoffset, GLsizei width, GLsizei height, GLsizei depth, GLenum format, GLsizei imageSize, ArrayBufferView data);
Taking this as a follow-on to the other recent texture work.
Changing synopsis to reflect functions being implemented under this bug.
Created attachment 404287 [details] Patch
Comment on attachment 404287 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=404287&action=review > Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:1311 > + synthesizeGLError( > + GraphicsContextGL::INVALID_OPERATION, "compressedTexImage2D", > + "a buffer is bound to PIXEL_UNPACK_BUFFER"); We typically wouldn't split this over lines, but it's ok.
Committed r264377: <https://trac.webkit.org/changeset/264377> All reviewed patches have been landed. Closing bug and clearing flags on attachment 404287 [details].
<rdar://problem/65568996>
Comment on attachment 404287 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=404287&action=review >> Source/WebCore/html/canvas/WebGL2RenderingContext.cpp:1311 >> + "a buffer is bound to PIXEL_UNPACK_BUFFER"); > > We typically wouldn't split this over lines, but it's ok. Understood - this was to make emacs' auto-indenting be compatible with WebKit's style. Otherwise it would auto-indent the second line to the open parenthesis, leading to a style error.