Skip to content

Commit

Permalink
Remove wpt/svg/linking/reftests/url-processing-invalid-base.svg
Browse files Browse the repository at this point in the history
This test appears to be based on non-standard URL behavior. It assumes
that a relative URL fragment like "#fragment" combined with the base URL
"invalid:" results in an invalid URL. However, this is not standard
compliant behavior. The combination of a relative fragment and the valid
base URL should always result in a valid URL.

Note: 'valid' (or 'invalid' ) in the context of this CL's description
means what the URL Standard defines here:

https://url.spec.whatwg.org/#valid-url-string

In this sense, "invalid:" is a valid base URL.

This test is currently passing on Chromium, but failing on Safari and
Firefox.

https://wpt.fyi/results/svg/linking/reftests/url-processing-invalid-base.svg?label=experimental&label=master&aligned

If Chromium supports non-special URLs correctly
(https://crbug.com/40063064), this test will fail on Chromium too.

Since any valid base URL cannot be made invalid by any relative
fragment, removing this test seems reasonable.

This test was added in https://crrev.com/c/2115650.

Bug: 40621475,40063064
Change-Id: I46dee043ab02e7899a512e5a2cee3fc773848e5c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5399999
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1281665}
  • Loading branch information
hayatoito authored and chromium-wpt-export-bot committed Apr 3, 2024
1 parent ce7e878 commit e46be99
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions svg/linking/reftests/url-processing-invalid-base.svg

This file was deleted.

0 comments on commit e46be99

Please # to comment.