Closed
Bug 810252
Opened 12 years ago
Closed 11 years ago
remove pango text-shaping codepath
Categories
(Core :: Graphics: Text, defect)
Tracking
()
RESOLVED
FIXED
mozilla24
People
(Reporter: jfkthame, Assigned: karlt)
References
Details
Attachments
(1 file)
53.52 KB,
patch
|
jfkthame
:
review+
|
Details | Diff | Splinter Review |
Now that we've enabled harfbuzz for all text shaping on Linux (in bug 797398), we can remove the older code path that uses pango to shape text runs, and simplify the Linux font backend accordingly. (This is somewhat tied up with the fontconfig-based font matching code, which we'll still need to use, so it may not be an entirely simple excision.) I don't propose doing this immediately; we should leave the old code in place for a while until bug 797398 has shipped to the release channel (it's currently in mozilla19) and we're confident in the results.
Reporter | ||
Comment 1•11 years ago
|
||
It's been several cycles since we flipped the pref; I think it's time to say goodbye to the codepath we're no longer using.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → karlt
Assignee | ||
Comment 2•11 years ago
|
||
We won't be removing the dependency on libpango as gtk+ will pull in this library anyway and we'll need to keep some pango code to get system font metrics from gtk+.
Summary: remove pango text-shaping codepath and eliminate dependency on libpango → remove pango text-shaping codepath
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #753072 -
Flags: review?(jfkthame)
Reporter | ||
Comment 4•11 years ago
|
||
Can we also drop the use of PangoLanguage (and just use our gecko language codes to label fontsets where needed), or is that not feasible at the moment?
Assignee | ||
Comment 5•11 years ago
|
||
Font selection (gfxPangoFontGroup) still uses pango_language_includes_script() which uses mPangoLanguage. Perhaps there are some other parts of the code that can be moved from a PangoLanguage to nsIAtom, since Gecko language atoms are (sometimes or always?) now languages instead of langGroups, but that requires more thought and is independent from the shaping code.
Reporter | ||
Comment 6•11 years ago
|
||
Fair enough; let's just do this for now. Thanks!
Reporter | ||
Updated•11 years ago
|
Attachment #753072 -
Flags: review?(jfkthame) → review+
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/03be241006e0
Flags: in-testsuite-
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/03be241006e0
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in
before you can comment on or make changes to this bug.
Description
•