538103 - font fullnames (e.g. Arial Bold) are matched incorrectly as font family names

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[John Daggett (:jtd)]

Under Windows we match "Arial Bold" to Arial, rather than using the fallback font.  This is an artefact of GDI which allows either in lookups.

Steps:

1. Load URL

Result: some red entries appear in Arial

Expected result: black enteries appear in Arial, red in Georgia

Firefox matches IE, it maps Arial Bold to Arial but doesn't draw using the bold face.  One might argue "meh, good enough" but this is completely inconsistent with the behavior on other platforms.

Note: inconsistencies in other browsers

Opera10/Mac: matches Arial Bold to the bold face of Arial
Opera10/Win: does not match Arial Bold

Webkit/Mac: matches Postscript name Arial-BoldMT to bold face of Arial but does not match Arial Bold
Webkit/Win: matches Arial Bold to the bold face of Arial but does not match Arial-BoldMT

(Webkit = Safari4/Chrome4)

Comment 1

[Jonathan Kew [:jfkthame]]

Your testcase works as intended for me using a build that includes (updated versions of) patches from bugs 493280 and 502906. (The significant one would be bug 493280.) Marking this as dependent.

Comment 2

[Jonathan Kew [:jfkthame]]

Attachment: collection of reftests based on the Arial testcase

Here are a group of reftests based on the original testcase, split into sections to make it easier to pinpoint failures.

It's difficult to test this stuff with 100% certainty, in the absence of a known, controlled collection of installed fonts, but these tests ought to detect the incorrect behavior on typical Mac and Windows systems, at least.

If Arial fonts are not installed at all (e.g. on Linux systems, perhaps?), the tests are intended to pass harmlessly, simply using generic fonts. Checking with tryserver to confirm this behavior.

Comment 3

[John Daggett (:jtd)]

Comment on attachment 421287 [details] [diff] [review]
collection of reftests based on the Arial testcase

Looks good but I would use only the family names Arial and Georgia (with this casing, not quoted) in the ref pages.

Comment 4

[Jonathan Kew [:jfkthame]]

http://hg.mozilla.org/mozilla-central/rev/2b6846eef536