Closed Bug 1622229 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 22236 - Correctly test document.all quirks

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox76 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla76
Tracking Flags:
Tracking Status
firefox76 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Attachments

(4 files)

Notable wpt changes
804.45 KB, text/plain
Details
Notable wpt changes
804.45 KB, text/plain
Details
Notable wpt changes
804.45 KB, text/plain
Details
Notable wpt changes
804.45 KB, text/plain
Details

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

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

Aryeh Gregor <ayg@aryeh.name> wrote:

Correctly test document.all quirks

For legacy compatibility, this property is specified with some willful
violations of JS: https://html.spec.whatwg.org/#dom-document-all.
Prior to this commit, idlharness wasn't accounting for these, so
html/dom/interfaces.html was wrong for document.all/HTMLAllCollection
tests.

It's a bit ugly to include this directly in idlharness.js and
testharness.js, but it's the cleanest solution I can think of.

Fixes #4207.

Attached file Notable wpt changes —
# CI Results Ran 13 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI Total 291 tests ## Status Summary ### Firefox OK : 224[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 225[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt, Gecko-windows7-32-opt] 226[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug, Gecko-windows7-32-debug] 227[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 245[GitHub] PASS : 21781[Gecko-windows7-32-opt] 21783[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 21811[Gecko-windows7-32-debug] 21813[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 21883[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 21892[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 23745[GitHub] FAIL : 3847[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 4019[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 4021[Gecko-windows7-32-opt] 4038[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 4040[Gecko-windows7-32-debug] 4048[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 6996[GitHub] TIMEOUT: 2 ### Chrome OK : 245 PASS : 29194 FAIL : 1624 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e312c9cd0915714e444c3b4631348147e42c3b1) [GitHub PR Head](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes —
# CI Results Ran 13 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI Total 291 tests ## Status Summary ### Firefox OK : 224[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 225[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt, Gecko-windows7-32-opt] 226[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug, Gecko-windows7-32-debug] 227[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 245[GitHub] PASS : 21781[Gecko-windows7-32-opt] 21783[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 21811[Gecko-windows7-32-debug] 21813[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 21883[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 21892[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 23745[GitHub] FAIL : 3847[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 4019[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 4021[Gecko-windows7-32-opt] 4038[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 4040[Gecko-windows7-32-debug] 4048[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 6996[GitHub] TIMEOUT: 2 ### Chrome OK : 245 PASS : 29194 FAIL : 1624 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e312c9cd0915714e444c3b4631348147e42c3b1) [GitHub PR Head](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_base) ## Details (See attachment for full changes)
Attached file Notable wpt changes —
# CI Results Ran 13 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI Total 291 tests ## Status Summary ### Firefox OK : 224[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 225[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt, Gecko-windows7-32-opt] 226[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug, Gecko-windows7-32-debug] 227[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 245[GitHub] PASS : 21781[Gecko-windows7-32-opt] 21783[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 21811[Gecko-windows7-32-debug] 21813[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 21883[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 21892[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 23745[GitHub] FAIL : 3847[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 4019[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 4021[Gecko-windows7-32-opt] 4038[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 4040[Gecko-windows7-32-debug] 4048[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 6996[GitHub] TIMEOUT: 2 ### Chrome OK : 245 PASS : 29194 FAIL : 1624 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e312c9cd0915714e444c3b4631348147e42c3b1) [GitHub PR Head](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_base) ## Details (See attachment for full changes)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/65eedd9b56ee [wpt PR 22236] - Correctly test document.all quirks, a=testonly https://hg.mozilla.org/integration/autoland/rev/1109fcdb0f36 [wpt PR 22236] - Update wpt metadata, a=testonly
Attached file Notable wpt changes —
# CI Results Ran 13 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI Total 291 tests ## Status Summary ### Firefox OK : 224[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 225[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt, Gecko-windows7-32-opt] 226[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug, Gecko-windows7-32-debug] 227[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 245[GitHub] PASS : 21781[Gecko-windows7-32-opt] 21783[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 21811[Gecko-windows7-32-debug] 21813[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 21883[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 21892[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 23745[GitHub] FAIL : 3847[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview] 4019[Gecko-windows10-64-opt, Gecko-windows10-64-qr-opt] 4021[Gecko-windows7-32-opt] 4038[Gecko-windows10-64-debug, Gecko-windows10-64-qr-debug] 4040[Gecko-windows7-32-debug] 4048[Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt] 6996[GitHub] TIMEOUT: 2 ### Chrome OK : 245 PASS : 29194 FAIL : 1624 ## Links [Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e312c9cd0915714e444c3b4631348147e42c3b1) [GitHub PR Head](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_head) [GitHub PR Base](https://wpt.fyi/results/?sha=f3584b66101c34602c4c1c957610a303e36107cb&label=pr_base) ## Details (See attachment for full changes)
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/65eedd9b56ee
https://hg.mozilla.org/mozilla-central/rev/1109fcdb0f36

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox76: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: