RESOLVED FIXED 223425
Add WebGL 1.0.4 and 2.0.1 test files to the repository
https://bugs.webkit.org/show_bug.cgi?id=223425
Summary Add WebGL 1.0.4 and 2.0.1 test files to the repository
Kimmo Kinnunen
Reported 2021-03-18 01:43:24 PDT
Add WebGL 1.0.4 and 2.0.1 test files to the repository The import of the test suites is too big commit for the infrastructure to handle, so import partially.
Attachments
Patch (19.16 MB, patch)
2021-03-18 01:55 PDT, Kimmo Kinnunen
ews-feeder: commit-queue-
Kimmo Kinnunen
Comment 1 2021-03-18 01:55:33 PDT
Kenneth Russell
Comment 2 2021-03-18 13:17:13 PDT
Not 100% sure how this was intended to work, but I see that the files are being put into LayoutTests/webgl/resources/webgl_test_files. This generally seems fine though note you won't be able to version these, so the monikers "1.0.4" and "2.0.1" won't have the effect you intend. Maybe better to name those 1.0.x and 2.0.x to better show they're not strictly versioned. Will try to mark this r+, though having trouble doing so in the tool.
Kenneth Russell
Comment 3 2021-03-18 13:17:25 PDT
+James.
EWS
Comment 4 2021-03-19 03:57:50 PDT
Committed r274711: <https://commits.webkit.org/r274711> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423574 [details].
Radar WebKit Bug Importer
Comment 5 2021-03-19 03:58:20 PDT
Aakash Jain
Comment 6 2021-03-22 07:16:42 PDT
(In reply to EWS from comment #4) > Committed r274711: <https://commits.webkit.org/r274711> webgl/conformance/extensions/khr-parallel-shader-compile.html seems to be consistently failing on iOS after this change. History: https://results.webkit.org/?suite=layout-tests&test=webgl%2Fconformance%2Fextensions%2Fkhr-parallel-shader-compile.html
Kimmo Kinnunen
Comment 8 2021-03-22 08:04:28 PDT
(In reply to Aakash Jain from comment #6) > (In reply to EWS from comment #4) > > Committed r274711: <https://commits.webkit.org/r274711> > webgl/conformance/extensions/khr-parallel-shader-compile.html seems to be > consistently failing on iOS after this change. Thanks! Strange, I thought I ran the iOS-simulator for that dir.. I can add that to the skip list if nobody else does it before, but only tomorrow..
Note You need to log in before you can comment on or make changes to this bug.