1645268 - New wpt failures in /html/dom/usvstring-reflection.https.html

Status: NEW

(Core :: DOM: Core & HTML, defect)

Description

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Syncing wpt PR 24118 found new untriaged test failures in CI

Tests Affected

Firefox-only failures

/html/dom/usvstring-reflection.https.html
source : unpaired surrogate codepoint should be replaced with U+FFFD: FAIL
anchor : unpaired surrogate codepoint should be replaced with U+FFFD: FAIL
area : unpaired surrogate codepoint should be replaced with U+FFFD: FAIL
storage event : unpaired surrogate codepoint should be replaced with U+FFFD: FAIL

New Tests That Don't Pass

/html/dom/usvstring-reflection.https.html
UnregisterProtocolHandler URL: unpaired surrogate codepoint should not make any exceptions.: FAIL (Chrome: PASS, Safari: FAIL)

CI Results

Gecko CI (Treeherder)
GitHub PR Head

Notes

These updates will be on mozilla-central once bug 1645131 lands.

Note: this bug is for tracking fixing the issues and is not
owned by the wpt sync bot.

This bug is linked to the relevant tests by an annotation in
https://github.com/web-platform-tests/wpt-metadata. These annotations
can be edited using the wpt interop dashboard
https://jgraham.github.io/wptdash/

If this bug is split into multiple bugs, please also update the
annotations, otherwise we are unable to track which wpt issues are
already triaged. Resolving as duplicate or closing this issue should
be cause the bot to automatically update or remove the annotation.

Comment 1

[Thomas Wisniewski [:twisniewski]]

We can pass all but one of the failures trivially by updating some WebIDL DOMStrings to USVStrings to match the specs, and fixing an error in the WPT itself where it doesn't send an options parameter to createDataChannelPair.

Then we'll only fail the last case because we don't yet support unregisterProtocolHandler (bug 1631477).

I've got a patch to do this which I'll land after the patch in bug 1867680.

Comment 2

[Thomas Wisniewski [:twisniewski]]

Attachment: Bug 1645268 - update source.src|srcset, area.ping, a.ping, and RTCDataChannelsend to use USVString instead of DOMString to match the specs. Also fix a WPT bug so it lets us pass the RTC test; r?smaug

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/08a247571a8c
update source.src|srcset, area.ping, a.ping, and RTCDataChannelsend to use USVString instead of DOMString to match the specs. Also fix a WPT bug so it lets us pass the RTC test; r=smaug

Comment 4

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/45829 for changes under testing/web-platform/tests

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/08a247571a8c

Comment 6

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot