Closed
Bug 797398
Opened 12 years ago
Closed 12 years ago
use harfbuzz for all text shaping on Linux
Categories
(Core :: Layout: Text and Fonts, defect)
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(2 files)
(deleted),
patch
|
jtd
:
review+
|
Details | Diff | Splinter Review |
(deleted),
image/jpeg
|
Details |
Now that harfbuzz supports Indic shaping, we should consider using it for all text on Linux, in preference to the old pango codepath. According to Behdad, new harfbuzz is now as good as or better than old pango, and is shipping in stable GNOME.
Once we shape all text through harfbuzz, we'll be able to rip out a bunch of old, crufty code that supports the pango backend, but we should leave it in place (just change the harfbuzz.scripts pref) for a cycle or two so that there's an easy way to revert behavior in case of any unexpected issues.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #669137 -
Flags: review?(jdaggett)
Assignee | ||
Comment 2•12 years ago
|
||
(Patch is based on top of bug 797402.) Tryserver run at https://tbpl.mozilla.org/?tree=Try&rev=6a8bf71053e2.
Updated•12 years ago
|
Attachment #669137 -
Flags: review?(jdaggett) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Assignee: nobody → jfkthame
Target Milestone: --- → mozilla19
Comment 4•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
This patch _greatly_ improved Firefox's Khmer script display under linux. The old pango engine had false-positives and/or too willing to insert dotted-circles on broken clusters, inserting such glyph on script that would render ok on windows / osx.
Yay.
Comment 6•12 years ago
|
||
For the record, thought I'd attach a visual comparison of what was broken before switching to harfbuzz-ng indic rendering. Once again, thanks for the improvement. For Cambodia, this is certainly newsworthy :)
Comment 7•12 years ago
|
||
This patch removed the "(except when using AAT fonts)" comment (for Mac). The comment had just been added in bug 797402. Intentional?
Assignee | ||
Comment 8•12 years ago
|
||
Not really. I just pushed a followup to restore it:
https://hg.mozilla.org/integration/mozilla-inbound/rev/35f68c5fae83
Thanks for noticing!
Comment 9•12 years ago
|
||
Hmm, Jonathan, I've spotted a pretty big rendering regression with Khmer script when using harfbuzz with a specific font set. A bug has been filed on freedesktop.org: https://bugs.freedesktop.org/show_bug.cgi?id=55827
Long story short, vowels are misplaced and erroneously duplicated. Should I file a bug over here too? This will definitively require synching with harfbuzz when it's fixed as it greatly hampers reading of khmer script.
Comment 10•12 years ago
|
||
(I'm assuming this also affects khmer script rendering on OSX)
Assignee | ||
Comment 11•12 years ago
|
||
Yes, please file a new bug for us to update harfbuzz again once that issue is fixed upstream. Thanks!
Comment 12•12 years ago
|
||
Jonathan, filed bug 799869.
Comment 13•12 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•