Closed Bug 1988818 Opened 4 months ago Closed 4 months ago

[wpt-sync] Sync PR 54848 - [ReferenceTarget] Make IDL attribute nullable

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
145 Branch
Tracking Status
firefox145 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Attachments

(1 file)

Sync web-platform-tests PR 54848 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/54848
Details from upstream follow.

Daniel Clark <daniec@microsoft.com> wrote:

[ReferenceTarget] Make IDL attribute nullable

Per https://github.com/WICG/webcomponents/issues/1093 the
referenceTarget IDL attribute should be a nullable DOMString.
Update the types in the Blink implementation accordingly.
Update the reference-target-basic.html WPT that's affected by this.
Add coverage to the reference target WPTs for aria-labeledby,
commandfor, interestfor, and popovertarget to validate that setting
referenceTarget to null makes idrefs target the host, and setting
referenceTarget to the empty string makes them invalid.

Bug: 346835896
Change-Id: I7699d7ebcb9bf50db15a54336ad0acf05379f71c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6943596
Commit-Queue: Dan Clark \<daniec@microsoft.com>
Reviewed-by: Alice Boxhall \<alice@igalia.com>
Reviewed-by: Mason Freed \<masonf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1515789}

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Attached file Notable wpt changes
# CI Results Ran 9 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI Total 13 tests and 6 subtests ## Status Summary ### Firefox `OK` : 5[GitHub] 11[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt] `PASS` : 1[GitHub] 4520[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt] `FAIL` : 19[GitHub] 1681[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-24h2-debug, Gecko-windows11-32-24h2-opt, Gecko-windows11-64-24h2-debug, Gecko-windows11-64-24h2-opt] `ERROR`: 2 ### Chrome `OK` : 5 `PASS` : 8 `FAIL` : 12 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1224e51279ed7dbaa907d8bd1184e4cc5a5cbd95) [GitHub PR Head](https://wpt.fyi/results/?sha=aa02f1332a39f463f92667bbe2ea668a3810ca22&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=aa02f1332a39f463f92667bbe2ea668a3810ca22&label=pr_base) ## Details (See attachment for full changes)
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 145 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: