Bug 466956: fix alias check in gfxFontconfigUtils::ResolveFontName for correct return value, r=karlt, sr=roc
authorPeter Weilbacher <mozilla@weilbacher.org>
Tue, 02 Dec 2008 20:29:17 +0100
changeset 22170 6156d0a3976339f2ad48e45f91168e3bc05f9a4b
parent 22169 f6fde414896b459adf0d478d45e74d082b3f84d0
child 22171 67f8a5b061561662c14e4e19b45dc237e0294e26
push id3838
push usermozilla@weilbacher.org
push dateTue, 02 Dec 2008 19:30:46 +0000
treeherdermozilla-central@6156d0a39763 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskarlt, roc
bugs466956
milestone1.9.2a1pre
Bug 466956: fix alias check in gfxFontconfigUtils::ResolveFontName for correct return value, r=karlt, sr=roc
gfx/thebes/src/gfxFontconfigUtils.cpp
--- a/gfx/thebes/src/gfxFontconfigUtils.cpp
+++ b/gfx/thebes/src/gfxFontconfigUtils.cpp
@@ -731,17 +731,17 @@ gfxFontconfigUtils::ResolveFontName(cons
     // check just whether the font is existing, without resolving to regular
     // name.
     //
     // The family names in mAliasForMultiFonts are names understood by
     // fontconfig.  The actual font to which they resolve depends on the
     // entire match pattern.  That info is not available here, but there
     // will be a font so leave the resolving to the gfxFontGroup.
     if (IsExistingFamily(fontname) ||
-        mAliasForMultiFonts.IndexOfIgnoreCase(fontname))
+        mAliasForMultiFonts.IndexOfIgnoreCase(fontname) != -1)
         aAborted = !(*aCallback)(aFontName, aClosure);
 
     return NS_OK;
 }
 
 PRBool
 gfxFontconfigUtils::IsExistingFamily(const nsCString& aFamilyName)
 {