Closed Bug 1953818 Opened 5 months ago Closed 5 months ago

[wpt-sync] Sync PR 51322 - Check whether an element is custom in the spec-compliant way

Categories

(Core :: CSS Parsing and Computation, task, P4)

task

Tracking

()

RESOLVED FIXED
138 Branch
Tracking Status
firefox138 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Xiaocheng Hu <xiaochengh.work@gmail.com> wrote:

Check whether an element is custom in the spec-compliant way

Current code checks whether an element is custom by checking whether it has an associated custom element definition. This is different from the spec, which says:

An element whose custom element state is "custom" is said to be custom.

The two definitions are identical in almost all cases, except during custom element upgrade, where the custom element state is set to failed until finished.

As a result, the current implementation may erroneously fire custom element callbacks for mutations during the upgrade.

This PR fixes it.

Reviewed in servo/servo#35960

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core

CI Results

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

Total 1 tests and 6 subtests

Status Summary

Firefox

OK : 1
PASS: 6

Chrome

OK : 1
PASS: 6

Safari

OK : 1
PASS: 6

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a6be449c446b [wpt PR 51322] - Check whether an element is custom in the spec-compliant way, a=testonly
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in before you can comment on or make changes to this bug.