Commit 3431fc81 authored by msaboff@apple.com's avatar msaboff@apple.com
Browse files

2011-06-11 Michael Saboff <msaboff@apple.com>

        Reviewed by James Robinson.

        REGRESSION(88260): 10-50% performance regression across many page cyclers
        https://bugs.webkit.org/show_bug.cgi?id=62349

        Made CHROMIUM platform specific values for cMaxInactiveFontData and
        cTargetInactiveFontData to mitigate the current performance issue on Chromium
        page cycler tests.  Made the chromium value for cMaxInactiveFontData 250 based
        on recommendation from James Robinson.  Changed cTargetInactiveFontData to 200
        so that 50 inactive fonts will be cleaned up at a time.

        Change made to address existing Chromium performance test regression.

        * platform/graphics/FontCache.cpp:


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@88611 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 031254e0
2011-06-11 Michael Saboff <msaboff@apple.com>
Reviewed by James Robinson.
REGRESSION(88260): 10-50% performance regression across many page cyclers
https://bugs.webkit.org/show_bug.cgi?id=62349
Made CHROMIUM platform specific values for cMaxInactiveFontData and
cTargetInactiveFontData to mitigate the current performance issue on Chromium
page cycler tests. Made the chromium value for cMaxInactiveFontData 250 based
on recommendation from James Robinson. Changed cTargetInactiveFontData to 200
so that 50 inactive fonts will be cleaned up at a time.
Change made to address existing Chromium performance test regression.
* platform/graphics/FontCache.cpp:
2011-06-11 Adam Barth <abarth@webkit.org>
Reviewed by Darin Adler.
......
......@@ -251,8 +251,13 @@ typedef HashMap<FontPlatformData, pair<SimpleFontData*, unsigned>, FontDataCache
static FontDataCache* gFontDataCache = 0;
const int cMaxInactiveFontData = 120; // Pretty Low Threshold
const int cTargetInactiveFontData = 100;
#if PLATFORM(CHROMIUM)
const int cMaxInactiveFontData = 250;
const int cTargetInactiveFontData = 200;
#else
const int cMaxInactiveFontData = 50; // Pretty Low Threshold
const int cTargetInactiveFontData = 30;
#endif
static ListHashSet<const SimpleFontData*>* gInactiveFontData = 0;
SimpleFontData* FontCache::getCachedFontData(const FontDescription& fontDescription, const AtomicString& family, bool checkingAlternateName, ShouldRetain shouldRetain)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment