reftests gfx/layers/apz/reftests/async-scrollbar-1-v.html show a discrepancy in the scroll position on android since freetype 2.8

RESOLVED FIXED in Firefox 67

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
7 days ago

People

(Reporter: jfkthame, Assigned: kats)

Tracking

57 Branch
mozilla67
All
Android
Points:
---

Firefox Tracking Flags

(firefox55 unaffected, firefox56 unaffected, firefox57 wontfix, firefox67 fixed)

Details

(Whiteboard: [gfx-noted])

Reporter

Description

2 years ago
See bug 1364714 comment 42. The freetype 2.8 update causes a (small) change in the font metrics returned for our default sans-serif font on Android, and for some reason this appears to be causing a discrepancy on these reftests.

It's not immediately clear to me why font metrics should be important here, though, as the tests don't seem to use any text, just a div with a (huge) specified height.
Also for the record, when adding those tests I recall running into a lot of rounding errors and the numbers that I used for the scrollbar position and size of the div were chosen precisely to avoid much of the rounding error. It's quite likely that the freetype update just perturbed enough stuff to make those particular numbers now trigger a different rounding error so the scrollbar position is off by one. We can probably just adjust the numbers a little to bring the scrollbar position back in alignment.
OS: Unspecified → Android
Priority: -- → P3
Hardware: Unspecified → All
Whiteboard: [gfx-noted]
Version: unspecified → 57 Branch

In bug 1392147 it looks like the fuzz amount went back down. I think we can close this.

Assignee: nobody → kats

Comment 3

4 months ago
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ec32c653cca2
Drop reference to outstanding issue since the issue was resolved. r=me and DONTBUILD

Comment 4

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.