389697 - remove freetype2 font code

Status: RESOLVED FIXED

(Core Graveyard :: GFX, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

I have a patch to remove some additional code that's no longer used now that cairo is the only option.  This is the freetype2 font code, which allowed Mozilla to find fonts using freetype2 (if the user configured Mozilla to tell it where to look, which meant it wasn't particularly useful).

This patch also some default values for preferences that were used by that code, and some preferences that were used by other pre-Xft X11 font code.

Note that we still need the configure check for the freetype2 library since cairo needs it.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Er, which allowed Mozilla to display fonts using freetype2 (if the user configured Mozilla to tell it where to find the fonts, which meant it wasn't particularly useful).

Comment 2

[Mats Palmgren (inactive)]

FYI, I think there are few more places that should be fixed:
http://lxr.mozilla.org/seamonkey/search?string=MOZ_ENABLE_FREETYPE2

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

This addresses Mats's comments (removes the 2 files in gfx/idl and the line in autoconf.mk.in).

I think we can probably also remove nsFontList/nsIFontList (although I'm not sure), but that would be a pretty sizable patch, so should be separate.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Oops, I forgot to exclude the configure diffs from the new patch, so it's much bigger than it needs to be...

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

And another followup should be removing MOZ_ENABLE_XFT, I think...

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Checked in to trunk.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Filed bug 397813 on nsFontList/nsIFontList.