It's possible for us to update these tests to use Python, but given that these tests were imported from blink, and given that we declined to import failing test, it seems better to simple delete the tests which rely on PHP.
<rdar://problem/78401190>
Created attachment 429535 [details] Patch
The tests being removed: imported/blink/http/tests/css/remove-placeholder-styles.html imported/blink/http/tests/security/referrer-policy-attribute-img-origin-when-crossorigin.html imported/blink/http/tests/security/referrer-policy-attribute-img-origin.html imported/blink/http/tests/security/referrer-policy-attribute-img-picture.html imported/blink/http/tests/security/referrer-policy-attribute-img-removeattr.html imported/blink/http/tests/security/referrer-policy-attribute-img-unsafe-url.html imported/blink/http/tests/svg/image-fetch-fail.svg
(In reply to Jonathan Bedard from comment #3) > The tests being removed: > imported/blink/http/tests/css/remove-placeholder-styles.html No replacement, but never worked the way the author thought it did anyways (PHP is the css directory is totally inert) > imported/blink/http/tests/security/referrer-policy-attribute-img-origin-when-crossorigin.html > imported/blink/http/tests/security/referrer-policy-attribute-img-origin.html > imported/blink/http/tests/security/referrer-policy-attribute-img-picture.html > imported/blink/http/tests/security/referrer-policy-attribute-img-removeattr.html > imported/blink/http/tests/security/referrer-policy-attribute-img-unsafe-url.html Much more thorough implementations of this in http/tests/referrer-policy-img > imported/blink/http/tests/svg/image-fetch-fail.svg Looks to be covered in more detail by imported/w3c/web-platform-tests/svg
(In reply to Jonathan Bedard from comment #4) > (In reply to Jonathan Bedard from comment #3) > > The tests being removed: > > imported/blink/http/tests/css/remove-placeholder-styles.html > > No replacement, but never worked the way the author thought it did anyways > (PHP is the css directory is totally inert) Oh, wait, never mind, in http/tests
(In reply to Jonathan Bedard from comment #5) > (In reply to Jonathan Bedard from comment #4) > > (In reply to Jonathan Bedard from comment #3) > > > The tests being removed: > > > imported/blink/http/tests/css/remove-placeholder-styles.html > > > > No replacement, but never worked the way the author thought it did anyways > > (PHP is the css directory is totally inert) > > Oh, wait, never mind, in http/tests imported/blink/http/tests/css/remove-placeholder-styles.html looks to be a duplicate of http/tests/css/link-css-disabled-value-with-slow-loading-sheet.html
Committed r278092 (238167@main): <https://commits.webkit.org/238167@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 429535 [details].