Closed Bug 1740922 Opened 3 years ago Closed 3 years ago

High frequency [tier 2] layout/reftests/bugs/1738700-1.html == layout/reftests/bugs/1738700-1-ref.html | image comparison, max difference: 3, number of differing pixels: 1707 Gecko 96 merges to Beta on 2021-12-06

Categories

(Core :: Layout, defect)

defect

Tracking

()

VERIFIED FIXED
96 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox94 --- unaffected
firefox95 --- unaffected
firefox96 + verified

People

(Reporter: ctuns, Assigned: jfkthame)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

[Tracking Requested - why for this release]:

Central-as-late beta simulation:
How to run these simulations:

Failure log:
Reftest analyzer link:

''' INFO '''

Many of the arabic characters are highlighted as different from the reference image.

''' INFO '''

[task 2021-11-12T13:09:54.710Z] 13:09:54     INFO -  REFTEST TEST-START | layout/reftests/bugs/1738700-1.html == layout/reftests/bugs/1738700-1-ref.html
[task 2021-11-12T13:10:05.830Z] 13:09:54     INFO -  REFTEST TEST-LOAD | http://10.7.205.220:8854/tests/layout/reftests/bugs/1738700-1.html | 67 / 2169 (3%)
[task 2021-11-12T13:10:05.830Z] 13:10:05     INFO -  REFTEST TEST-LOAD | http://10.7.205.220:8854/tests/layout/reftests/bugs/1738700-1-ref.html | 67 / 2169 (3%)
[task 2021-11-12T13:10:05.830Z] 13:10:05     INFO -  REFTEST INFO | REFTEST fuzzy test (0, 0) <= (3, 1707) <= (3, 1328)
[task 2021-11-12T13:10:05.830Z] 13:10:05  WARNING -  REFTEST TEST-UNEXPECTED-FAIL | layout/reftests/bugs/1738700-1.html == layout/reftests/bugs/1738700-1-ref.html | image comparison, max difference: 3, number of differing pixels: 1707

Hi Jonathan, your latest patch from bug 1740413 doesn't seem to work. This is still happening on beta sim, we got 1 green from 10. Can you please take a look? Thanks.

Flags: needinfo?(jkew)

(In reply to Cristian Tuns from comment #1)

Hi Jonathan, your latest patch from bug 1740413 doesn't seem to work. This is still happening on beta sim, we got 1 green from 10. Can you please take a look? Thanks.

Looks like the fuzz limit needs to be raised further. The glyphs are all rendering in the right places, there's just some discrepancy in the antialiased edges.

(It's frustrating that there seems to be so much variability between runs; do we know why this is? Are we running these Android test jobs on a variety of slightly different configurations picked at random from among a pool of machines? Otherwise, I don't see why we should see such inconsistency in the number of fuzzy antialiasing pixels.)

Flags: needinfo?(jkew)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED

Seems like we get a more-or-less arbitrary number of fuzz-pixels, but as long as the max-difference is only 3, there's no real problem here; an actual misplaced glyph would result in a much greater difference and would be clearly visible as red in the screenshot.

Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f6de6f4ead4
Increase antialiasing fuzz again to allow for varying results on Android. r=lsalzman
See Also: → 1741047
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: