Some object-fit WPT tests need a fuzzy annotation (rather than fails)
Categories
(Core :: Layout, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox70 | --- | fixed |
People
(Reporter: dholbert, Assigned: dholbert)
References
Details
Attachments
(1 file)
We have some object-fit-*-png-*
WPT tests that are annotated as failing (on non-webrender) due to a fuzzy image-scaling issue. This is basically fine -- the tests are only expected to be pixel-perfect if they can get image-rendering: pixelated
, which we don't yet support.
Since it's not too many pixels differing, we should just annotate these as fuzzy rather than failing, to better reflect reality & to get a bit more strictness.
Assignee | ||
Comment 1•5 years ago
|
||
Here's a Try run that shows the failures for object-fit-cover-png-001 and -002 (after I've removed their .ini files):
https://treeherder.mozilla.org/#/jobs?repo=try&revision=38d8022f5b3470804452b32673095bd216aa862f
Assignee | ||
Comment 2•5 years ago
|
||
Another Try run, with a few more .ini files removed/tweaked, and with bespoke fuzzy annotations based on that ^^ run combined with a local ./mach wpt
run in this directory:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ec1b7bcebb538399339e3246dec9c09168b5f6f0
(I anticipate that this run will have some orange due to differences between platforms.)
Assignee | ||
Comment 3•5 years ago
|
||
All of these tests current fail due to fuzzy upscaling/downscaling behavior,
which the tests aim to work around using image-rendering: pixelated
, but we
don't yet support that feature. I've added the bug link for that feature to
these .ini files, so that we'll get a clue to remove these annotations once we
fix that bug and implement that feature.
Assignee | ||
Comment 4•5 years ago
|
||
Updated Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=277cd1e490fffcb67b359d2af210d60061ea76bd
Still waiting on Android test runs, but everything else is green.
Assignee | ||
Comment 5•5 years ago
|
||
Try run with Android annotations added:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8333e0ed5daa36441fe3146a608e34672c7f9bbe
Looks good, gonna go ahead and land.
Updated•5 years ago
|
Comment 7•5 years ago
|
||
bugherder |
Description
•