Bug 185876 – Do not convert testharness paths in import-w3c-tests

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Chris Nardi]

Do not convert testharness paths in import-w3c-tests

[Chris Nardi]

Created attachment 340996 [details]
Patch

[youenn fablet]

Note that some WebKittens would actually like to convert all testharness paths so that some of these tests could also be run file based.

[Chris Nardi]

(In reply to youenn fablet from comment #2)
> Note that some WebKittens would actually like to convert all testharness
> paths so that some of these tests could also be run file based.

Right, I forgot about all of that. Bringing us closer to upstream rather than further seems preferable, as it would make it easier to share modifications back and forth. But it's probably not you I need to convince of that. :)

[Alex Christensen]

Comment on attachment 340996 [details]
Patch

This has been requesting review for more than one year.  If this is still needed, please rebase and re-request review.

[Sam Sneddon [:gsnedders]]

(In reply to youenn fablet from comment #2)
> Note that some WebKittens would actually like to convert all testharness
> paths so that some of these tests could also be run file based.

We've had all the code to do this since 2014 (bug 134763), and yet we've never turned it on for WPT (and it was explicitly disabled from day one for WPT in bug 134764). We _did_ have this turned on for the old CSS WG test suite, prior to it getting merged into WPT; this was removed six years ago in bug 170647.

As such, unless people are manually invoking the importer with this (and presumably any such import would simply get trampled at the next re-import), this code has been entirely dead since 2017 and we should just get rid of it.

There's a real cost to rewriting tests (especially in terms of enabling export) and it seems unlikely we're ever going revert course here and start rewriting WPT.

[Sam Sneddon [:gsnedders]]

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

[EWS]

Committed 265842@main (1866a4e1cf48): <https://commits.webkit.org/265842@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/111907873>