Closed Bug 1675871 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 26438 - Quirks: Add tests for "html fills viewport" and "body fills html"

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

b'Andreu Botella <abb@randomunok.com>' wrote:

Quirks: Add tests for "html fills viewport" and "body fills html"

The "html fills viewport" quirk wasn't tested at all; and the tests for "body fills html" only covered the exceptions to the quirk rather than the cases where it should apply. This PR adds the missing coverage.

Component: web-platform-tests → Layout
Product: Testing → Core

CI Results

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

Total 7 tests

Status Summary

Firefox

PASS: 3
FAIL: 4

Chrome

PASS: 7

Safari

PASS: 5
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/quirks/html-fills-viewport-quirk-vertical.html: FAIL
/quirks/html-fills-viewport-quirk.html: FAIL

New Tests That Don't Pass

/quirks/body-fills-html-quirk-vertical.html: FAIL (Chrome: PASS, Safari: FAIL)
/quirks/body-fills-html-quirk.html: FAIL (Chrome: PASS, Safari: FAIL)
/quirks/html-fills-viewport-quirk-vertical.html: FAIL (Chrome: PASS, Safari: PASS)
/quirks/html-fills-viewport-quirk.html: FAIL (Chrome: PASS, Safari: PASS)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/97e4d1a3a7a0
[wpt PR 26438] - Quirks: Add tests for "html fills viewport" and "body fills html", a=testonly
https://hg.mozilla.org/integration/autoland/rev/dc670f01d7f6
[wpt PR 26438] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.