Bug 252210

Summary:

New test: [iOS / macOS] webgl/webgl-ext-norm16-texture-texsubimage-nocrash.html consistently failing / crashing

Product:

WebKit

Reporter:

Ryan Haddad <ryanhaddad>

Component:

WebGL

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

dino, kbr, kkinnunen, webkit-bot-watchers-bugzilla, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

Other

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

252237

Bug Blocks:

Attachments:

Description	Flags
crash log	none

Description Ryan Haddad 2023-02-13 16:25:22 PST

webgl/webgl-ext-norm16-texture-texsubimage-nocrash.html has been consistently failing on macOS and iOS release bots and crashing on debug since it was added with https://commits.webkit.org/260114@main

https://results.webkit.org/?suite=layout-tests&test=webgl%2Fwebgl-ext-norm16-texture-texsubimage-nocrash.html

--- /Volumes/Data/worker/Apple-Ventura-Release-AppleSilicon-WK2-Tests/build/layout-test-results/webgl/webgl-ext-norm16-texture-texsubimage-nocrash-expected.txt
+++ /Volumes/Data/worker/Apple-Ventura-Release-AppleSilicon-WK2-Tests/build/layout-test-results/webgl/webgl-ext-norm16-texture-texsubimage-nocrash-actual.txt
@@ -1,8 +1,8 @@
-Test that texSubImage3D does not crash on with EXT_norm16_texture.
+Test that texSubImage does not crash on with EXT_norm16_texture.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
-TEST COMPLETE: 97 PASS, 0 FAIL
+TEST COMPLETE: 93 PASS, 4 FAIL
 
 Test: 2d R16_EXT RED UNSIGNED_SHORT
 PASS getError was expected value: NO_ERROR :
@@ -38,11 +38,11 @@
 PASS Test: RG16_SNORM_EXT RG UNSIGNED_SHORT did not crash.
 Test: 2d RGB16_EXT RGB UNSIGNED_SHORT
 PASS getError was expected value: NO_ERROR :
-PASS getError was expected value: NO_ERROR :
+FAIL getError expected: NO_ERROR. Was INVALID_OPERATION :
 PASS Test: RGB16_EXT RGB UNSIGNED_SHORT did not crash.
 Test: 2d RGB16_SNORM_EXT RGB SHORT
 PASS getError was expected value: NO_ERROR :
-PASS getError was expected value: NO_ERROR :
+FAIL getError expected: NO_ERROR. Was INVALID_OPERATION :
 PASS Test: RGB16_SNORM_EXT RGB SHORT did not crash.
 Test: 2d RGB16_EXT RGB SHORT
 PASS getError was expected value: INVALID_OPERATION :
@@ -102,11 +102,11 @@
 PASS Test: RG16_SNORM_EXT RG UNSIGNED_SHORT did not crash.
 Test: 3d RGB16_EXT RGB UNSIGNED_SHORT
 PASS getError was expected value: NO_ERROR :
-PASS getError was expected value: NO_ERROR :
+FAIL getError expected: NO_ERROR. Was INVALID_OPERATION :
 PASS Test: RGB16_EXT RGB UNSIGNED_SHORT did not crash.
 Test: 3d RGB16_SNORM_EXT RGB SHORT
 PASS getError was expected value: NO_ERROR :
-PASS getError was expected value: NO_ERROR :
+FAIL getError expected: NO_ERROR. Was INVALID_OPERATION :
 PASS Test: RGB16_SNORM_EXT RGB SHORT did not crash.
 Test: 3d RGB16_EXT RGB SHORT
 PASS getError was expected value: INVALID_OPERATION :

Comment 1 Radar WebKit Bug Importer 2023-02-13 16:25:33 PST

<rdar://problem/105422460>

Comment 2 Ryan Haddad 2023-02-13 16:27:58 PST

Created attachment 464977 [details]
crash log

Thread 5 Crashed:: RemoteGraphicsContextGL work queue
0   libANGLE-shared.dylib         	       0x13e6a5580 gl::LogMessage::~LogMessage() + 240 (debug.cpp:208)
1   libANGLE-shared.dylib         	       0x13e6a4e55 gl::LogMessage::~LogMessage() + 21 (debug.cpp:182)
2   libANGLE-shared.dylib         	       0x13efa7c78 gl::ValidImageDataSize(gl::Context const*, angle::EntryPoint, gl::TextureType, int, int, int, unsigned int, unsigned int, void const*, int) + 376 (validationES.cpp:1256)
3   libANGLE-shared.dylib         	       0x13efd884b gl::ValidateES3TexImageParametersBase(gl::Context const*, angle::EntryPoint, gl::TextureTarget, int, unsigned int, bool, bool, int, int, int, int, int, int, int, unsigned int, unsigned int, int, void const*) + 4379 (validationES3.cpp:797)
4   libANGLE-shared.dylib         	       0x13efd9244 gl::ValidateES3TexImage2DParameters(gl::Context const*, angle::EntryPoint, gl::TextureTarget, int, unsigned int, bool, bool, int, int, int, int, int, int, int, unsigned int, unsigned int, int, void const*) + 340 (validationES3.cpp:903)
5   libANGLE-shared.dylib         	       0x13efc8fc5 gl::ValidateTexSubImage2DRobustANGLE(gl::Context const*, angle::EntryPoint, gl::TextureTarget, int, int, int, int, int, unsigned int, unsigned int, int, void const*) + 645 (validationES2.cpp:2809)
6   libANGLE-shared.dylib         	       0x13e79fc1b GL_TexSubImage2DRobustANGLE + 219 (entry_points_gles_ext_autogen.cpp:2171)
7   WebCore                       	       0x17462a7fe WebCore::GraphicsContextGLANGLE::texSubImage2D(unsigned int, int, int, int, int, int, unsigned int, unsigned int, GCGLSpan<void const, 18446744073709551615ul>) + 190 (GraphicsContextGLANGLE.cpp:415)

Comment 3 Kimmo Kinnunen 2023-02-14 10:07:51 PST

Fixed in bug 252237

*** This bug has been marked as a duplicate of bug 252237 ***