Closed Bug 1749414 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 32319 - Remove modal-dialog-scroll-height.html

Categories

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

task

Tracking

()

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Joey Arhar <jarhar@chromium.org> wrote:

Remove modal-dialog-scroll-height.html

This test is failing in all browsers on wpt.fyi:
https://wpt.fyi/results/html/semantics/interactive-elements/the-dialog-element/modal-dialog-scroll-height.html

I'm not sure why it passes in the chromium test infrastructure, but
people want to remove it:
https://github.com/web-platform-tests/interop-2022/issues/12#issuecomment-1009154280

Steve, I see you added this test here:
https://codereview.chromium.org/574143002
Do you have any ideas for how else we could write this test which
doesn't check the explicit height? Maybe with a reftest? I couldn't
easily understand the bug and the change.

Change-Id: If27a4b07ca235190d19cbe2f52bbfe6dd2594225

Reviewed-on: https://chromium-review.googlesource.com/3379191
WPT-Export-Revision: 197074e369ab2c378256b2c26a6882ed7830879a

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in before you can comment on or make changes to this bug.