Closed Bug 1622229 Opened 8 months ago Closed 8 months ago

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

Categories

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

task

Tracking

(firefox76 fixed)

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Attachments

(4 files)

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.
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.