improve bitmap font support (FreeType only)

src/ports/SkFontConfigInterface_direct.cpp

Index: src/ports/SkFontConfigInterface_direct.cpp
===================================================================
--- src/ports/SkFontConfigInterface_direct.cpp	(revision 12480)
+++ src/ports/SkFontConfigInterface_direct.cpp	(working copy)
@@ -330,11 +330,11 @@
 }
 
 static bool valid_pattern(FcPattern* pattern) {
-    FcBool is_scalable;
-    if (FcPatternGetBool(pattern, FC_SCALABLE, 0, &is_scalable) != FcResultMatch
-        || !is_scalable) {
-        return false;
-    }
+//    FcBool is_scalable;

[reed1, 2013/12/09 16:58:42]

Why is this section commented out? Can we delete it?

[bungeman-skia, 2013/12/09 19:42:44]

This change is required so that bitmap fonts are not rejected outright on Linux.
With this code still in place, users of this path (which is currently Skia tools
and Chromium) will never see a bitmap font. This is currently like this due to
the uncertainty of simply enabling these in Chrome. Seeing that the layout tests
pass with this change, it seems like we might be ok to remove this.

+//    if (FcPatternGetBool(pattern, FC_SCALABLE, 0, &is_scalable) != FcResultMatch
+//        || !is_scalable) {
+//        return false;
+//    }
 
     // fontconfig can also return fonts which are unreadable
     const char* c_filename = get_name(pattern, FC_FILE);