1705876 - [wpt-sync] Sync PR 28559 - Remove Elements from toJSON

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

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

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

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

b'Toshiaki Tanaka <[e-mail]>' wrote:

Remove Elements from toJSON

1. PerformanceElementTiming
2. LargestContentfulPaint
3. LayoutShiftAttribution

These include element or node in each toJSONForBinding implementation.

Those should be removed because they use the default toJSON operation and
those should not include interfaces that themselves do not have a toJSON
operation. So we should remove them from the toJSON builder and update
the existing toJSON WPT tests,

Bug: 1199954
Change-Id: I282439c17485264f1621102e3d002a693d48c1f9
Reviewed-on: https://chromium-review.googlesource.com/2831166
WPT-Export-Revision: 63a333d2d05b1bf4ae0a6304affb970d37aa9f6a

Comment 1

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=b67160b7eb2abd6a075c368737a789dc24ab05dd

Comment 2

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

CI Results

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

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
FAIL: 2

Chrome

OK : 2
FAIL: 2

Safari

OK : 2
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/element-timing/toJSON.html
Test toJSON() in PerformanceElementTiming.: FAIL (Chrome: FAIL, Safari: FAIL)
/largest-contentful-paint/toJSON.html
Test toJSON() in LargestContentfulPaint.: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bfc27327c565
[wpt PR 28559] - Remove Elements from toJSON, a=testonly

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/bfc27327c565