Bug 1118981 - initialize mSkipDrawing correctly for already loading fonts. r=jfkthame, a=sylvestre
authorJohn Daggett <jdaggett@mozilla.com>
Mon, 19 Jan 2015 09:00:26 +0900
changeset 242901 beb62e1ad523
parent 242900 dfbca180664d
child 242902 fe217a0d2e9a
push id4334
push userjdaggett@mozilla.com
push date2015-01-19 00:01 +0000
treeherdermozilla-beta@beb62e1ad523 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjfkthame, sylvestre
bugs1118981
milestone36.0
Bug 1118981 - initialize mSkipDrawing correctly for already loading fonts. r=jfkthame, a=sylvestre
gfx/thebes/gfxTextRun.cpp
--- a/gfx/thebes/gfxTextRun.cpp
+++ b/gfx/thebes/gfxTextRun.cpp
@@ -1696,17 +1696,19 @@ gfxFontGroup::FindPlatformFont(const nsA
             family = mUserFontSet->LookupFamily(aName);
             if (family) {
                 nsAutoTArray<gfxFontEntry*,4> userfonts;
                 family->FindAllFontsForStyle(mStyle, userfonts, needsBold);
                 // add these to the fontlist
                 uint32_t count = userfonts.Length();
                 for (uint32_t i = 0; i < count; i++) {
                     fe = userfonts[i];
-                    mFonts.AppendElement(FamilyFace(family, fe, needsBold));
+                    FamilyFace ff(family, fe, needsBold);
+                    ff.CheckState(mSkipDrawing);
+                    mFonts.AppendElement(ff);
                 }
             }
         }
     }
 
     // Not known in the user font set ==> check system fonts
     if (!family) {
         gfxPlatformFontList *fontList = gfxPlatformFontList::PlatformFontList();