Bug 139628

Summary: Remove FontCachePurgePreventer
Product: WebKit Reporter: Antti Koivisto <koivisto>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mmaxfield, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch andersca: review+

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+
Antti Koivisto
Comment 1 2015-01-11 14:42:22 PST
Antti Koivisto
Comment 2 2015-01-11 14:58:50 PST
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.