RESOLVED FIXED Bug 42185
Synchronize Khronos and WebKit WebGL tests
https://bugs.webkit.org/show_bug.cgi?id=42185
Summary Synchronize Khronos and WebKit WebGL tests
Kenneth Russell
Reported 2010-07-13 13:03:19 PDT
Many of the WebGL layout tests which originated in WebKit have been adopted as portions of the Khronos WebGL conformance suite. It is desirable to consider the Khronos repository the master version of these tests, because multiple vendors are making improvements to them in that repository. Recent changes to the WebKit layout tests have been merged into the Khronos version of the tests. At this point we need to synchronize the Khronos versions of the WebKit WebGL layout tests back to the WebKit repository, to get back to a state where the tests are identical (modulo some paths of .js includes which are different in the two repositories). This synchronization will be done in a few stages because it will touch a lot of test code. This bug will be used to track the progress.
Attachments
Patch (156.76 KB, patch)
2010-07-15 11:46 PDT, Kenneth Russell
kbr: commit-queue-
Revised patch (157.22 KB, patch)
2010-07-15 11:53 PDT, Kenneth Russell
japhet: review+
kbr: commit-queue-
Zhenyao Mo
Comment 1 2010-07-13 13:16:55 PDT
(In reply to comment #0) > Many of the WebGL layout tests which originated in WebKit have been adopted as portions of the Khronos WebGL conformance suite. It is desirable to consider the Khronos repository the master version of these tests, because multiple vendors are making improvements to them in that repository. Recent changes to the WebKit layout tests have been merged into the Khronos version of the tests. At this point we need to synchronize the Khronos versions of the WebKit WebGL layout tests back to the WebKit repository, to get back to a state where the tests are identical (modulo some paths of .js includes which are different in the two repositories). > > This synchronization will be done in a few stages because it will touch a lot of test code. This bug will be used to track the progress. Maybe we should change the folder structure in khronos conformance tests to match the WebKit Layout tests. That way we can do a full sync with very little effort.
Kenneth Russell
Comment 2 2010-07-13 13:38:03 PDT
(In reply to comment #1) > Maybe we should change the folder structure in khronos conformance tests to match the WebKit Layout tests. That way we can do a full sync with very little effort. This would require the creation of some unnecessary directories in the Khronos test suite. I think that we can get the same effect with a simple shell script to perform the synchronization -- which I will probably add while fixing this bug.
Kenneth Russell
Comment 3 2010-07-15 11:46:44 PDT
Created attachment 61695 [details] Patch From the ChangeLog: For the WebGL tests in the Khronos repository that originated in the WebKit repository, synchronized the latest versions of these tests using a script which will be checked in later. Copied webgl-test-utils.js from Khronos repo and added needed entry points to webgl-test.js. Rebaselined tests as necessary and verified the rebaselining manually.
Kenneth Russell
Comment 4 2010-07-15 11:53:51 PDT
Created attachment 61698 [details] Revised patch Replaced tabs with spaces in Khronos repo in a couple of tests, and re-synchronized those tests.
Kenneth Russell
Comment 5 2010-07-15 12:13:16 PDT
WebKit Review Bot
Comment 6 2010-07-15 13:38:06 PDT
http://trac.webkit.org/changeset/63444 might have broken Leopard Intel Release (Tests)
Note You need to log in before you can comment on or make changes to this bug.