Bug 243819 – webgl/2.0.y/deqp/functional/gles3/negativetextureapi.html assertion failure

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

[Kimmo Kinnunen]

webgl/2.0.y/deqp/functional/gles3/negativetextureapi.html assertion failure

The validation is not run when pixels == nullptr

Due to bug 242614

[Kimmo Kinnunen]

<rdar://98347913>

[Kimmo Kinnunen]

The issue is that validation is not done when user passes pixels=null, where as the teximage3dbase expect that the validation is done.

WebGL defines pixels=null means pixels=sufficiently large 0 filled pixels.

The philosophical rabbit hole:
Current WebGL implementation naturally "optimizes" so that it doesn't create the sufficiently large 0 filled pixels buffer, rather that it just passes pixels=nullptr down.

However, it also validates other arguments based on pixels. The validation shoudn't be done this way, it should be done based on the GL type, since pixels isn't always there.

The other philosophical rabbit hole:
the ASSERT_UNUSED(ok) is the smell that reveals the bug. Instead, the validation should return the computed properties, if the validation passes. This way the computation is done once

[Dan Glastonbury]

Pull request: https://github.com/WebKit/WebKit/pull/3263

[EWS]

Committed 253543@main (592cd219941f): <https://commits.webkit.org/253543@main>

Reviewed commits have been landed. Closing PR #3263 and removing active labels.