Closed Bug 1626246 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 22562 - [css-flexbox] Rename flexbox-overflow-auto-* tests to overflow-auto-*

Categories

(Core :: Layout: Flexbox, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Mario Sanchez Prada <mario@igalia.com> wrote:

[css-flexbox] Rename flexbox-overflow-auto-* tests to overflow-auto-*

This fix normalizes the current situation where we have these files:

wpt/css/css-flexbox/flexbox-overflow-auto-001.html
wpt/css/css-flexbox/flexbox-overflow-auto-002.html
wpt/css/css-flexbox/overflow-auto-001.html
wpt/css/css-flexbox/overflow-auto-002.html
wpt/css/css-flexbox/overflow-auto-003.html
wpt/css/css-flexbox/overflow-auto-004.html

...into a more consistent file naming scheme for this type of tests:

wpt/css/css-flexbox/overflow-auto-001.html
wpt/css/css-flexbox/overflow-auto-002.html
wpt/css/css-flexbox/overflow-auto-003.html
wpt/css/css-flexbox/overflow-auto-004.html
wpt/css/css-flexbox/overflow-auto-005.html
wpt/css/css-flexbox/overflow-auto-006.html

Bug: 1063749
Change-Id: I5d73bb4d54ee536a8a1894428cb557ea1be95f3e
Reviewed-on: https://chromium-review.googlesource.com/2129531
WPT-Export-Revision: 0a6d12983fd3050f88a8ad3a4b8193fea68fe8a6

PR 22562 applied with additional changes from upstream: 997e66ba5837e2ee7dc2d16ff678c6265cea418e
Component: web-platform-tests → Layout: Flexbox
Product: Testing → Core

CI Results

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

Total 50 tests

Status Summary

Firefox

OK : 1
PASS: 4[GitHub] 52[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt]
FAIL: 13

Chrome

OK : 1
PASS: 17

Safari

OK : 1
PASS: 10
FAIL: 7

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/css/css-flexbox/overflow-auto-006.html
.flexbox, .inline-flexbox 11: FAIL
.flexbox, .inline-flexbox 2: FAIL
.flexbox, .inline-flexbox 1: FAIL
.flexbox, .inline-flexbox 6: FAIL
.flexbox, .inline-flexbox 5: FAIL
.flexbox, .inline-flexbox 12: FAIL

New Tests That Don't Pass

/css/css-flexbox/overflow-auto-005.html: FAIL (Chrome: PASS, Safari: FAIL)
/css/css-flexbox/overflow-auto-006.html
.flexbox, .inline-flexbox 11: FAIL (Chrome: PASS, Safari: PASS)
.flexbox, .inline-flexbox 10: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 3: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 2: FAIL (Chrome: PASS, Safari: PASS)
.flexbox, .inline-flexbox 1: FAIL (Chrome: PASS, Safari: PASS)
.flexbox, .inline-flexbox 14: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 6: FAIL (Chrome: PASS, Safari: PASS)
.flexbox, .inline-flexbox 5: FAIL (Chrome: PASS, Safari: PASS)
.flexbox, .inline-flexbox 4: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 13: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 9: FAIL (Chrome: PASS, Safari: FAIL)
.flexbox, .inline-flexbox 12: FAIL (Chrome: PASS, Safari: PASS)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7d6987a99ee5 [wpt PR 22562] - [css-flexbox] Rename flexbox-overflow-auto-* tests to overflow-auto-*, a=testonly https://hg.mozilla.org/integration/autoland/rev/48ce89132357 [wpt PR 22562] - Update wpt metadata, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77

:aciure, note that overflow-auto-006.html has be fixed in WPT, so that in the next sync it should pass on Firefox fully.

https://github.com/web-platform-tests/wpt/pull/22574
https://github.com/web-platform-tests/wpt/pull/22688

You need to log in before you can comment on or make changes to this bug.