Closed Bug 1000745 Opened 7 years ago Closed 7 years ago
Remove obsolete font
Extracting that part from bug 947654...
Assignee: nobody → fred.wang
Status: NEW → ASSIGNED
Attachment #8411618 - Flags: review?(karlt)
Is "font.mathfont-glyph-tables" also obsolete? There's no entry by that name in all.js.
This does not change the style context.
Sorry to mix up with the patches. Hopefully that version is the right one.
(In reply to John Daggett (:jtd) away 25 - 29 april from comment #2) > Is "font.mathfont-glyph-tables" also obsolete? There's no entry by that name > in all.js. So reading again, I see that font.mathfont-glyph-tables is only used with EnumerateFamilies to register the private glyph tables. I think we can just remove it and do calls with hardcoded family-name gGlyphTableList->AddGlyphTable("MathJax_Main") gGlyphTableList->AddGlyphTable("STIXNonUnicode") etc until we remove deprecate these fonts and remove these old tables.
This patch removes font.mathfont-glyph-tables and should be yet another perf improvement, since we no longer need to read the mathfont.property file at init. That seems to work for me (tested with STIX & MathJax). My only concern is if EnumerateFamilies does some magic to determine the family string (I'm assuming it only splits the list).
Attachment #8411676 - Flags: review?(roc) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.