Closed
Bug 1392259
Opened 5 years ago
Closed 3 years ago
reftests gfx/layers/apz/reftests/async-scrollbar-1-v.html show a discrepancy in the scroll position on android since freetype 2.8
Categories
(Core :: Panning and Zooming, enhancement, P3)
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox55 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | wontfix |
firefox67 | --- | fixed |
People
(Reporter: jfkthame, Assigned: kats)
Details
(Whiteboard: [gfx-noted])
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.
Assignee | ||
Comment 1•5 years ago
|
||
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
Assignee | ||
Updated•5 years ago
|
status-firefox55:
--- → unaffected
status-firefox56:
--- → unaffected
status-firefox57:
--- → fix-optional
Assignee | ||
Comment 2•3 years ago
|
||
In bug 1392147 it looks like the fuzz amount went back down. I think we can close this.
Assignee: nobody → kats
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•3 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Updated•3 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•