Changeset 132308 in webkit


Ignore:
Timestamp:
Oct 23, 2012 10:12:25 PM (11 years ago)
Author:
mitz@apple.com
Message:

The font cache evicts inactive font data too aggressively when not under memory pressure
https://bugs.webkit.org/show_bug.cgi?id=100194

Reviewed by Sam Weinig.

Increasing the number of inactive font data objects the cache may hold increases the hit
rate, leading to improved page load performance. When memory pressure is detected,
MemoryPressureHandler evicts all inactive objects, regardless of the limits we are
increasing here.

  • platform/graphics/FontCache.cpp:

(WebCore): Increased the maximum number of inactive font data objects in the cache from 50
to 225, and the number of objects to evict once the limit is met from 20 to 25.

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r132303 r132308  
     12012-10-23  Dan Bernstein  <mitz@apple.com>
     2
     3        The font cache evicts inactive font data too aggressively when not under memory pressure
     4        https://bugs.webkit.org/show_bug.cgi?id=100194
     5
     6        Reviewed by Sam Weinig.
     7
     8        Increasing the number of inactive font data objects the cache may hold increases the hit
     9        rate, leading to improved page load performance. When memory pressure is detected,
     10        MemoryPressureHandler evicts all inactive objects, regardless of the limits we are
     11        increasing here.
     12
     13        * platform/graphics/FontCache.cpp:
     14        (WebCore): Increased the maximum number of inactive font data objects in the cache from 50
     15        to 225, and the number of objects to evict once the limit is met from 20 to 25.
     16
    1172012-10-23  Takashi Sakamoto  <tasak@google.com>
    218
  • trunk/Source/WebCore/platform/graphics/FontCache.cpp

    r130968 r132308  
    294294const int cTargetInactiveFontData = 200;
    295295#else
    296 const int cMaxInactiveFontData = 50; // Pretty Low Threshold
    297 const int cTargetInactiveFontData = 30;
     296const int cMaxInactiveFontData = 225;
     297const int cTargetInactiveFontData = 200;
    298298#endif
    299299static ListHashSet<RefPtr<SimpleFontData> >* gInactiveFontData = 0;
Note: See TracChangeset for help on using the changeset viewer.