[webkit-dev] Supporting <link rel=> for finding ref tests
Simon Fraser
simon.fraser at apple.com
Fri Nov 8 14:00:04 PST 2019
I'd like to land a patch to support finding test references via <link rel="match/mismatch">:
https://bugs.webkit.org/show_bug.cgi?id=203784 <https://bugs.webkit.org/show_bug.cgi?id=203784>
There has been some discussion about this in the past:
https://lists.webkit.org/pipermail/webkit-dev/2011-November/018470.html <https://lists.webkit.org/pipermail/webkit-dev/2011-November/018470.html>
But I think the benefits outweigh the drawbacks. As that mail states:
> *Link element approach*
> Pros:
>
> - Can reuse same ref. file for multiple tests
Still true.
> - Can have multiple ref. files for single test
True but no something that we support, and I haven't see any WPT use this (our importer throws an error if it sees this)
> - Information is self-contained in the test file
Still true
> - We may get away with test suite build step
It certainly simplifies WPT test import.
Currently importing some CSS suites (e.g. css-backgrounds) results in broken -expected.html files because copying them breaks references to sub resources.
> (It turns out that we can't convert W3C ref tests to use WebKit conventions
> due to the first two points.)
We're doing this much more now, and the "multiple references" point is moot, so I think we can import WPT tests mostly as-is.
>
> Cons:
>
> - Requires us modifying each port's DRT to support this format
No, it just requires webkitpy hacking which I've done in the patch.
> - Adding link elements itself may affect tests (all W3C tests are
> required to have link elements at the moment)
I haven't seen this be an issue.
> - Hard to understand relationship between files. e.g. if we want to
> figure out which tests use ref.html, we must look at all test files
This is true, but I don't really see it being a problem in practice. What I have seen is us importing CSS 2.1 tests that have foo.html and foo-ref.html, and treating foo-ref.html as a test so generating foo-expected.txt and foo-ref-expected.txt. That seems worse.
> - Other browser vendors (Firefox and Opera) prefer manifest file approach
This is no longer true. "reftest.list" files are deprecated (webkit.org/b/203783, https://github.com/web-platform-tests/wpt/issues/20060 <https://github.com/web-platform-tests/wpt/issues/20060>).
So now that WPT is heavily invested in <link rel=> I think we should follow suite. It will simplify WPT import, and reduced the number of cloned -expected.html files significantly.
Simon
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20191108/f3f93124/attachment.htm>
More information about the webkit-dev
mailing list