Closed Bug 1762329 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 33441 - ZoomForDSF: Add an optional fuzzy factor to layout table checks + use it

Categories

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

Product:
Core
Component:
Layout: Tables
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
101 Branch
Tracking Flags:
Tracking Status
firefox101 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Vladimir Levin <vmpstr@chromium.org> wrote:

ZoomForDSF: Add an optional fuzzy factor to layout table checks + use it

This patch adds an optional data-fuzzy-factor attribute to
check-layout-th.js. If set, then this provides an error tolerance to
whatever parameter is being checked.

Typically there is only one parameter being checked, so there is no
confusion about what the fuzzy factor applies to, but if more than one
attribute is checked then the fuzzy factor affects them all.

This patch also adds the fuzzy factor check to a test to make it pass.

R=ikilpatrick@chromium.org

Fixed: 1309972
Change-Id: I662c88a50a645fe9b35b0083d9f44774393096ed
Reviewed-on: https://chromium-review.googlesource.com/3562227
WPT-Export-Revision: cfef55d3d4c84ba433507611ef6f7fe4ebd8ac75

Component: web-platform-tests → Layout: Tables
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 15 subtests

Status Summary

Firefox

OK : 1
PASS: 10
FAIL: 5

Chrome

OK : 1
PASS: 15

Safari

OK : 1
PASS: 10
FAIL: 5

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

/css/css-tables/tentative/table-width-redistribution-fixed-padding.html
table 8: FAIL linked bug:Bug 1711286

New Tests That Don't Pass

/css/css-tables/tentative/table-width-redistribution-fixed-padding.html
table 8: FAIL (Chrome: PASS, Safari: PASS)
table 11: FAIL (Chrome: PASS, Safari: FAIL)
table 12: FAIL (Chrome: PASS, Safari: FAIL)
table 13: FAIL (Chrome: PASS, Safari: FAIL)
table 15: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f2f0fc77542
[wpt PR 33441] - ZoomForDSF: Add fuzziness to a table width test, a=testonly

https://hg.mozilla.org/mozilla-central/rev/9f2f0fc77542

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox101: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
You need to log in before you can comment on or make changes to this bug.