Map Liberation Sans Mono and Ascender Sans Mono to Courier New....

skia/ext/SkFontHost_fontconfig_direct.cpp

 /* libs/graphics/ports/SkFontHost_fontconfig_direct.cpp
 **
 ** Copyright 2009, Google Inc.
 **
 ** Licensed under the Apache License, Version 2.0 (the "License");
 ** you may not use this file except in compliance with the License.
 ** You may obtain a copy of the License at
 **
 **     http://www.apache.org/licenses/LICENSE-2.0
 **
@@ skipping 15 matching lines @@
 
 namespace {
 
 // Equivalence classes, used to match the Liberation and Ascender fonts
 // with their metric-compatible replacements.  See the discussion in
 // GetFontEquivClass().
 enum FontEquivClass
 {
     OTHER,
     SANS,
-    SERIF
+    SERIF,
+    MONO
 };
 
 // Match the font name against a whilelist of fonts, returning the equivalence
 // class.
 FontEquivClass GetFontEquivClass(const char* fontname)
 {
     // It would be nice for fontconfig to tell us whether a given suggested
     // replacement is a "strong" match (that is, an equivalent font) or
     // a "weak" match (that is, fontconfig's next-best attempt at finding a
     // substitute).  However, I played around with the fontconfig API for
     // a good few hours and could not make it reveal this information.
     //
     // So instead, we hardcode.  Initially this function emulated
     //   /etc/fonts/conf.d/30-metric-aliases.conf
     // from my Ubuntu system, but we're better off being very conservative.
 
-    // "Ascender Sans" and "Ascender Serif" are the tentative names of
-    // another set of fonts metric-compatible with Arial and Times New Roman
-    // with a character repertoire much larger than Liberation.
+    // "Ascender Sans", "Ascender Serif" and "Ascender Sans Mono" are the
+    // tentative names of another set of fonts metric-compatible with
+    // Arial, Times New Roman and Courier New  with a character repertoire

[Evan Martin, 2010/06/14 23:08:55]

nit: you have an extra space before "with"

+    // much larger than Liberation. Note that Ascender Sans Mono/Liberation
+    // Sans Mono are metrically compatible with Courier New, but the former
+    // is sans-serif while ther latter is serif.
     if (strcasecmp(fontname, "Arial") == 0 ||
         strcasecmp(fontname, "Liberation Sans") == 0 ||
         strcasecmp(fontname, "Ascender Sans") == 0) {
         return SANS;
     } else if (strcasecmp(fontname, "Times New Roman") == 0 ||
                strcasecmp(fontname, "Liberation Serif") == 0 ||
                strcasecmp(fontname, "Ascender Serif") == 0) {
         return SERIF;
+    } else if (strcasecmp(fontname, "Courier New") == 0 ||
+               strcasecmp(fontname, "Liberation Sans Mono") == 0 ||

[Evan Martin, 2010/06/14 23:08:55]

I don't think we want to do this.

If you have:
  font-family: Courier New, monospace;
and the user configured some *other* font as their monospace font, this will
cause them to get Liberation Sans over their preference.

I think what we want to do is to map Courier New to a monospace-family font in
WebKit such that for "font-family: Courier New;" WebKit knows that it should try
the default monospace font in the fallback case.

Since Ascender is aesthetically compatible, I think that substitution is ok
though -- my objection is only to using Liberation here.

+               strcasecmp(fontname, "Ascender Sans Mono") == 0) {
+        return MONO;
     }
     return OTHER;
 }
 
 
 // Return true if |font_a| and |font_b| are visually and at the metrics
 // level interchangeable.
 bool IsMetricCompatibleReplacement(const char* font_a, const char* font_b)
 {
     FontEquivClass class_a = GetFontEquivClass(font_a);
@@ skipping 248 matching lines @@
 
 int FontConfigDirect::Open(unsigned fileid) {
     SkAutoMutexAcquire ac(mutex_);
     const std::map<unsigned, std::string>::const_iterator
         i = fileid_to_filename_.find(fileid);
     if (i == fileid_to_filename_.end())
         return -1;
 
     return open(i->second.c_str(), O_RDONLY);
 }