Bug 212271

<rdar://problem/63547935>

I'll investigate.

Can see that ToT WebKit on macOS passes the test, and that ToT WebKit on the simulator fails. Interestingly, ToT Chrome (Canary) fails this test, but passes Khronos' upstream version of the test: https://www.khronos.org/registry/webgl/sdk/tests/conformance/extensions/webgl-depth-texture.html Continuing to investigate. Likely, some translation of depth texture enums will have to occur inside ANGLE in order to run on iOS's ES 3.0 driver. Start of the output of the test on the iPad simulator: PASS WebGL context exists Testing binding enum with extension disabled PASS gl.texImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_ENUM. PASS gl.texImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_INT, null) generated expected GL error: INVALID_ENUM. PASS Successfully enabled WEBGL_depth_texture extension PASS WEBGL_depth_texture listed as supported and getExtension succeeded Testing WEBGL_depth_texture testing: UNSIGNED_SHORT PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_CUBE_MAP_POSITIVE_X, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_2D, 1, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, 1, 1, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, new Uint16Array(1)) generated expected GL error: INVALID_OPERATION. PASS gl.texImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, canvas2) generated one of expected GL errors: INVALID_VALUE or INVALID_ENUM or INVALID_OPERATION. PASS gl.copyTexImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, 0, 0, 1, 1, 0) generated one of expected GL errors: INVALID_ENUM or INVALID_OPERATION. FAIL gl.texImage2D(gl.TEXTURE_2D, 0, gl.DEPTH_COMPONENT, 8, 8, 0, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, null) expected: NO_ERROR. Was INVALID_OPERATION. PASS gl.texSubImage2D(gl.TEXTURE_2D, 0, 0, 0, 1, 1, gl.DEPTH_COMPONENT, gl.UNSIGNED_SHORT, new Uint16Array(1)) generated expected GL error: INVALID_OPERATION. FAIL gl.copyTexSubImage2D(gl.TEXTURE_2D, 0, 0, 0, 0, 0, 1, 1) expected: INVALID_OPERATION. Was INVALID_VALUE. PASS gl.generateMipmap(gl.TEXTURE_2D) generated expected GL error: INVALID_OPERATION. FAIL gl.checkFramebufferStatus(gl.FRAMEBUFFER) should be 36053. Was 36054. FAIL actual(0) < left(0) FAIL actual(0) < left(0) FAIL actual(0) < left(0) FAIL actual(0) < left(0) FAIL actual(0) < left(0) FAIL actual(0) < left(0) ...

The OpenGL ES driver's returning INVALID_OPERATION from the functions->texImage2D call inside TextureGL::setImageHelper: #0 0x00000006af309189 in rx::CheckError(gl::Context const*, char const*, char const*, char const*, unsigned int) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/renderergl_utils.cpp:1955 #1 0x00000006af3f776d in rx::TextureGL::setImageHelper(gl::Context const*, gl::TextureTarget, unsigned long, unsigned int, gl::Extents const&, unsigned int, unsigned int, unsigned char const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/TextureGL.cpp:254 #2 0x00000006af3f6900 in rx::TextureGL::setImage(gl::Context const*, gl::ImageIndex const&, unsigned int, gl::Extents const&, unsigned int, unsigned int, gl::PixelUnpackState const&, unsigned char const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/TextureGL.cpp:220 #3 0x00000006af3f09be in gl::Texture::setImage(gl::Context*, gl::PixelUnpackState const&, gl::TextureTarget, int, unsigned int, gl::Extents const&, unsigned int, unsigned int, unsigned char const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/Texture.cpp:1062 #4 0x00000006aef3be9e in gl::Context::texImage2D(gl::TextureTarget, int, int, int, int, int, unsigned int, unsigned int, void const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/Context.cpp:4008 #5 0x00000006aef3bfef in gl::Context::texImage2DRobust(gl::TextureTarget, int, int, int, int, int, unsigned int, unsigned int, int, void const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libANGLE/Context.cpp:4025 #6 0x00000006af05c97c in gl::TexImage2DRobustANGLE(unsigned int, int, int, int, int, int, unsigned int, unsigned int, int, void const*) at /Users/kbr/src/iOSWebKit/Source/ThirdParty/ANGLE/src/libGLESv2/entry_points_gles_ext_autogen.cpp:1187 #7 0x00000006aa6f2e3e in WebCore::ExtensionsGLANGLE::texImage2DRobustANGLE(unsigned int, int, int, int, int, int, unsigned int, unsigned int, int, void const*) at /Users/kbr/src/iOSWebKit/Source/WebCore/platform/graphics/angle/ExtensionsGLANGLE.cpp:352 The arguments to the call to the driver are: glTexImage2D(GL_TEXTURE_2D, /* level */ 0, GL_DEPTH_COMPONENT, 8, 8, /* border */ 0, GL_DEPTH_COMPONENT, GL_UNSIGNED_SHORT, nullptr);

Created attachment 400072 [details] Patch

Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE

Emulation of the unsized internal formats for depth/stencil textures - provided by the OES_depth_texture extension on ES 3.0 - on top of ES 3.0's sized internal formats for these textures seems required in ANGLE. The attached patch makes fast/canvas/webgl/webgl-depth-texture.html pass completely in the iOS Simulator. Will have to do more testing upstream in ANGLE to see whether it's completely correct on all platforms.

Committed r262074: <https://trac.webkit.org/changeset/262074> All reviewed patches have been landed. Closing bug and clearing flags on attachment 400072 [details].

ios-wk2 failure of webgl/1.0.3/conformance/extensions/webgl-depth-texture.html handled in follow-on Bug 212284, removing the failure expectations for this test on iOS.