Bug 205627 - REGRESSION (r253926): fast/canvas/webgl/array-bounds-clamping.html fails
Summary: REGRESSION (r253926): fast/canvas/webgl/array-bounds-clamping.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: James Darpinian
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-12-28 23:20 PST by Alexey Proskuryakov
Modified: 2020-02-12 11:42 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2019-12-28 23:20:04 PST
fast/canvas/webgl/array-bounds-clamping.html started to fail after switching to ANGLE in r253926

It got marked as expected failure in this revision for WebKit1, but not for WebKit2, so it now shows up as a failure on bots.

https://results.webkit.org/?suite=layout-tests&test=fast%2Fcanvas%2Fwebgl%2Farray-bounds-clamping.html

The failure is hardware specific - the test still passes on many Mac models. I didn't check very carefully, seems to be passing on AMD and failing on Intel or Nvidia.
Comment 1 Radar WebKit Bug Importer 2019-12-28 23:20:16 PST
<rdar://problem/58229090>
Comment 2 Kenneth Russell 2019-12-30 13:09:17 PST
Possible to temporarily mark the expected failure so that r253926 isn't rolled back? Might be an option that needs to be passed to ANGLE's internal instance of its shader compiler.
Comment 3 Dean Jackson 2020-01-03 02:27:45 PST
This was a tricky test to get right - I think it might be incorrect.

Our non-ANGLE code passed SH_CLAMP_INDIRECT_ARRAY_BOUNDS into the shader configuration (see Extensions3DOpenGLCommon::getTranslatedShaderSourceANGLE)

I don't see a way to do this from the EGL API, but maybe it isn't necessary? Does Chrome use this option on Apple hardware?
Comment 4 Kenneth Russell 2020-01-03 11:34:42 PST
It seems likely that ANGLE's use of this workaround needs to be expanded. Currently it looks like it's only enabled on Android and AMD hardware. May need to be done always on Mac.

https://source.chromium.org/chromium/chromium/src/+/master:third_party/angle/src/libANGLE/renderer/gl/renderergl_utils.cpp;l=1596;drc=34fdc18ec806aa05ed2029d723c31de0d141a742;bpv=1;bpt=1
Comment 5 James Darpinian 2020-01-28 16:52:11 PST
The workaround was enabled upstream, so this will be fixed in the next ANGLE roll.
Comment 6 James Darpinian 2020-02-12 11:42:43 PST
ANGLE was updated in r255782