419370 - allow single face font families via pref

Status: RESOLVED FIXED

(Core :: Graphics, defect, P3)

Description

[John Daggett (:jtd)]

On the mac, the monospace font has been typically defined to be Osaka-Mono.  Unfortunately, its family name is Osaka so there's no way to specify this specific face, only its family Osaka.  The solution for this is to allow a pref setting to specify a list of font faces that should be treated as separate families.  This allows Osaka-Mono to exist as a separate font family from Osaka.  This isn't something we want to encourage generally, but it's a simple workaround for dealing with this situation.

Also, move the list of font families whose name table is read in at startup out to a pref.

I've already worked on a patch for this for bug 390901 but that involves more interface work, this just separates out some of the back end work on the mac that needs to get done.

Comment 1

[John Daggett (:jtd)]

Requesting blocking since this work is needed to fix bug 390901, a 1.9 blocker.

Comment 2

[John Daggett (:jtd)]

Attachment: patch, v.0.3, allow single face font families via pref

This patch sets up code that looks for a pref setting 'font.preload-names-list' that lists individual fonts that should be treated as separate families.  This allows for backward-compatible support for Osaka-Mono.

Note: patch also includes cleanup code to move preloading of name tables into a pref setting and the trimming out of bogus italic faces for Helvetica, Courier.

Comment 3

[John Daggett (:jtd)]

Attachment: screenshot from ja locale user

This shows a comparison of the current trunk to a build with the patch.  With the patch ja locale users see (1) localized font family names and (2) Osaka Mono can be used for the monospace font.

Comment 4

[John Daggett (:jtd)]

One more note: with the patch there is still the problem of fonts with localized family names not displaying correctly for English locale users.  This is what will be fixed by the rest of 390901.

Comment 5

[John Daggett (:jtd)]

Attachment: patch, v.0.4, allow single face font families via pref, common pref reading

Moved the code for reading in a font list from a pref into a separate utility routine in gfxFontUtils.[h|cpp]

Comment 6

[John Daggett (:jtd)]

Attachment: patch, v.0.5, allow single face font families via pref, common pref reading

minor cleanups

Comment 7

[John Daggett (:jtd)]

checked in