RESOLVED FIXED 139628
Remove FontCachePurgePreventer 
https://bugs.webkit.org/show_bug.cgi?id=139628
Summary Remove FontCachePurgePreventer
Antti Koivisto
Reported 2014-12-14 12:39:51 PST
We should only purge font cache on top of the runloop. Then this stack helper can go away.
Attachments
patch (29.35 KB, patch)
2015-01-11 14:42 PST, Antti Koivisto 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2015-01-11 14:42:22 PST
Created attachment 244425 [details] patch
Antti Koivisto
Comment 2 2015-01-11 14:58:50 PST
https://trac.webkit.org/r178250
Antti Koivisto
Comment 3 2015-01-11 15:03:24 PST
This fix rdar://problem/19431581 which was 0 com.apple.WebCore 0x000000010a58cce0 WTF::Ref<WebCore::FontGlyphs>::get() + 16 1 com.apple.WebCore 0x000000010a5893e0 WebCore::pruneUnreferencedEntriesFromFontGlyphsCache() + 176 2 com.apple.WebCore 0x000000010a5995b2 WebCore::FontCache::purgeInactiveFontData(int) + 34 3 com.apple.WebCore 0x000000010a59946c WebCore::FontCache::purgeInactiveFontDataIfNeeded() + 156 4 com.apple.WebCore 0x0000000109ff962c WebCore::FontCache::enablePurging() + 108 5 com.apple.WebCore 0x0000000109ff95b9 WebCore::FontCachePurgePreventer::~FontCachePurgePreventer() + 25 6 com.apple.WebCore 0x0000000109ff6765 WebCore::FontCachePurgePreventer::~FontCachePurgePreventer() + 21 7 com.apple.WebCore 0x000000010b6d036a WebCore::RenderSVGText::removeChild(WebCore::RenderObject&) + 106 ... 83 com.apple.WebCore 0x000000010a589491 WebCore::pruneUnreferencedEntriesFromFontGlyphsCache() + 353 84 com.apple.WebCore 0x000000010a589c21 WebCore::retrieveOrAddCachedFontGlyphs(WebCore::FontDescription const&, WTF::PassRefPtr<WebCore::FontSelector>) + 1697
Ryosuke Niwa
Comment 4 2015-01-23 21:37:41 PST
Looks like this was 2.5% regression on PLT (see Mavericks results)?
Antti Koivisto
Comment 5 2015-01-24 08:48:10 PST
The regression was fixed in https://trac.webkit.org/r178673
Antti Koivisto
Comment 6 2015-01-24 08:49:41 PST
Unfortunately the bot was down so the step change from the commit is not visible.
Note You need to log in before you can comment on or make changes to this bug.