Closed Bug 1743200 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 31761 - Use proper cleanup in dialog-focusing-steps-inert.html

Categories

(Core :: DOM: Core & HTML, task, P4)

task

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Oriol Brufau <obrufau@igalia.com> wrote:

Use proper cleanup in dialog-focusing-steps-inert.html

The test was just cleaning up at the end of the test(), but the code was
not reached if some assert failed, making the following test() fail too.
The proper way to clean up is with add_cleanup().
This change makes the 2nd test pass in Firefox and Chromium.

PR 31761 applied with additional changes from upstream: 5a6f1ee2c25aa1f01ca09d853a2e5cb3b3a43ad6
Component: web-platform-tests → DOM: Core & HTML
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 2 subtests

Status Summary

Firefox

OK : 1
FAIL: 2

Chrome

OK : 1
PASS: 1
FAIL: 1

Safari

OK : 1
FAIL: 2

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/semantics/interactive-elements/the-dialog-element/dialog-focusing-steps-inert.html
dialog.show(): focusing steps should not change focus when dialog is inert: FAIL (Chrome: FAIL, Safari: FAIL)
dialog.showModal(): focusing steps should apply focus fixup rule when dialog is inert: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2f25181ea69e
[wpt PR 31761] - Use proper cleanup in dialog-focusing-steps-inert.html, a=testonly
https://hg.mozilla.org/integration/autoland/rev/fbfa84e24b69
[wpt PR 31761] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in before you can comment on or make changes to this bug.