Closed Bug 915938 Opened 9 years ago Closed 8 years ago
Inconsistent baseline for text in canvases
The baseline of the text in this canvas is inconsistent. I have seen this in other canvases as well.
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e3c19a339b36&tochange=5ceea82a79c7 Pref on SkiaGL looks suspicious. Though there are a number of Graphics changes in that regression range. Will do some additional bisecting.
Suspicion confirmed. The first bad revision is: changeset: 139475:bdd1a266176f user: James Willcox <email@example.com> date: Fri Jul 19 14:03:46 2013 -0400 summary: Bug 895409 - Enable SkiaGL canvas on Android r=blassey
On Galaxy Nexus I don't get any text at all in the top right box. Strange. We should fix this and get a mochitest in that covers it. George can you see if this is reproducible with the rebase. If so, please fix :)
Assignee: nobody → gwright
This is on a Nexus 7 2013 (Adreno 320) and a HTC One X+ (nVIDIA Tegra 3) both running Android 4.3.
This bug is due to hinting mismatches between layout and rendering. Basically, layout thinks we're unhinted (which we are), and skia thinks we're hinted (the default if no hint options are explicitly set). Grab our GlyphRenderingOptions from gfxPlatform in our canvas rendering context and pass it through to the DrawTarget in order to resolve this mismatch.
Attachment #817199 - Flags: review?(jfkthame)
Try run here: https://tbpl.mozilla.org/?tree=Try&rev=6d1cf6324942
Comment on attachment 817199 [details] [diff] [review] honour-hinting.patch Review of attachment 817199 [details] [diff] [review]: ----------------------------------------------------------------- Looks reasonable to me. :)
Attachment #817199 - Flags: review?(jfkthame) → review+
Comment on attachment 817199 [details] [diff] [review] honour-hinting.patch [Approval Request Comment] Regression caused by (bug #): 848491 User impact if declined: Canvas text won't render correctly on Android Testing completed (on m-c, etc.): Full try run, m-c Risk to taking this patch (and alternatives if risky): Low risk, only synchronises hinting preferences in different parts of the codebase String or IDL/UUID changes made by this patch: None
Comment on attachment 817199 [details] [diff] [review] honour-hinting.patch See previous comment. Accidentally flagged for release instead of aurora
Attachment #817199 - Flags: approval-mozilla-release? → approval-mozilla-aurora?
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Comment on attachment 817199 [details] [diff] [review] honour-hinting.patch Since this is low risk and a tracked regression, we'll take on branches. Thanks for getting this fix in.
Hrm, so it seems we're failing tests on beta/aurora because we fuzzed a bunch of tests on m-c that aren't fuzzed on beta/aurora. We should uplift the patches from bug 888446 as well
Had to increase the fuzz in text-not-in-doc-test.html a bit. https://hg.mozilla.org/releases/mozilla-aurora/rev/2769f814a0bb https://hg.mozilla.org/releases/mozilla-beta/rev/e2916694bdf2
You need to log in before you can comment on or make changes to this bug.