Closed Bug 1939936 Opened 2 months ago Closed 1 month ago

[wpt-sync] Sync PR 49911 - Ensure Element ID modifications inside disconnected shadow roots are registered

Categories

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

task

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Simon Wülker <simon.wuelker@arcor.de> wrote:

Ensure Element ID modifications inside disconnected shadow roots are registered

The previous code used Node::is_connected instead of Node::is_tree_connected, meaning that modifications to an element id inside a shadow root that is not connected to a document would not be registered.

Also fixes a crash with registering named elements caused by the same problem. Both of these changes are covered by the new web platform test.

We'll likely run into more of these issues as we integrate the shadow dom into the codebase.

try

Reviewed in servo/servo#34834

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
PASS: 1

Chrome

OK : 1
PASS: 1

Safari

OK : 1
PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/960fbe81f7e1 [wpt PR 49911] - Ensure Element ID modifications inside disconnected shadow roots are registered, a=testonly
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.