ABANDONED: linux: Avoid bitmaps for Cambria, Calibri, and Consolas.

ui/gfx/font_render_params_linux.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/font_render_params.h"
 
 #include <fontconfig/fontconfig.h>
 
 #include "base/command_line.h"
 #include "base/containers/mru_cache.h"
@@ skipping 94 matching lines @@
   }
 
   FcConfigSubstitute(NULL, pattern, FcMatchPattern);
   FcDefaultSubstitute(pattern);
   FcResult result;
   FcPattern* match = FcFontMatch(NULL, pattern, &result);
   FcPatternDestroy(pattern);
   if (!match)
     return false;
 
-  if (family_out) {
-    FcChar8* family = NULL;
-    FcPatternGetString(match, FC_FAMILY, 0, &family);
-    if (family)
-      family_out->assign(reinterpret_cast<const char*>(family));
-  }
+  FcChar8* family = NULL;
+  FcPatternGetString(match, FC_FAMILY, 0, &family);
+  if (family && family_out)
+    family_out->assign(reinterpret_cast<const char*>(family));
 
   if (params_out) {
     FcBool fc_antialias = 0;
     if (FcPatternGetBool(match, FC_ANTIALIAS, 0, &fc_antialias) ==
         FcResultMatch) {
       params_out->antialiasing = fc_antialias;
     }
 
     FcBool fc_autohint = 0;
     if (FcPatternGetBool(match, FC_AUTOHINT, 0, &fc_autohint) ==
@@ skipping 11 matching lines @@
     if (FcPatternGetBool(match, FC_HINTING, 0, &fc_hinting) == FcResultMatch) {
       int fc_hint_style = FC_HINT_NONE;
       if (fc_hinting)
         FcPatternGetInteger(match, FC_HINT_STYLE, 0, &fc_hint_style);
       params_out->hinting = ConvertFontconfigHintStyle(fc_hint_style);
     }
 
     int fc_rgba = FC_RGBA_NONE;
     if (FcPatternGetInteger(match, FC_RGBA, 0, &fc_rgba) == FcResultMatch)
       params_out->subpixel_rendering = ConvertFontconfigRgba(fc_rgba);
+
+    // Some Vista fonts ship with embedded bitmaps. Make sure we don't use them
+    // if antialiasing was requested: http://crbug.com/408059
+    if (params_out->antialiasing && family &&
+        (strcasecmp(reinterpret_cast<const char*>(family), "calibri") == 0 ||
+         strcasecmp(reinterpret_cast<const char*>(family), "cambria") == 0 ||
+         strcasecmp(reinterpret_cast<const char*>(family), "consolas") == 0)) {
+      params_out->use_bitmaps = false;
+    }
   }
 
   FcPatternDestroy(match);
   return true;
 }
 
 // Serialize |query| into a string and hash it to a value suitable for use as a
 // cache key.
 uint32 HashFontRenderParamsQuery(const FontRenderParamsQuery& query) {
   return base::Hash(base::StringPrintf("%d|%d|%d|%d|%s",
@@ skipping 69 matching lines @@
   synchronized_cache->cache.Clear();
 }
 
 #if defined(OS_CHROMEOS)
 void SetFontRenderParamsDeviceScaleFactor(float device_scale_factor) {
   device_scale_factor_for_internal_display = device_scale_factor;
 }
 #endif
 
 }  // namespace gfx