Changeset 107794 in webkit

Timestamp:

Feb 15, 2012 1:24:36 AM (12 years ago)

Author:

tonyg@chromium.org

Message:

[chromium] Don't use increased FontCache size on Android
​https://bugs.webkit.org/show_bug.cgi?id=78656

Reviewed by Adam Barth.

The chromium port uses a larger font cache size because it increases
performance on the intl1 and intl2 page cyclers. However, on Android
devices where resources are more constrained, it isn't desireable to
allow the FontCache to grow so big.

No new tests because no testable difference in functionality.

• platform/graphics/FontCache.cpp:

(WebCore):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/FontCache.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-02-15  Tony Gentilcore  <tonyg@chromium.org>
+
+        [chromium] Don't use increased FontCache size on Android
+        https://bugs.webkit.org/show_bug.cgi?id=78656
+
+        Reviewed by Adam Barth.
+
+        The chromium port uses a larger font cache size because it increases
+        performance on the intl1 and intl2 page cyclers. However, on Android
+        devices where resources are more constrained, it isn't desireable to
+        allow the FontCache to grow so big.
+
+        No new tests because no testable difference in functionality.
+
+        * platform/graphics/FontCache.cpp:
+        (WebCore):
+
 2012-02-15  Roland Steiner  <rolandsteiner@chromium.org>
 

trunk/Source/WebCore/platform/graphics/FontCache.cpp

@@ -253 +253 @@
 static FontDataCache* gFontDataCache = 0;
 
-#if PLATFORM(CHROMIUM)
+#if PLATFORM(CHROMIUM) && !OS(ANDROID)
 const int cMaxInactiveFontData = 250;
 const int cTargetInactiveFontData = 200;