Changeset 20069 in webkit

Timestamp:

Mar 8, 2007, 3:33:43 PM (18 years ago)

Author:

hyatt

Message:

Fix 2% performance regression on the PLT. Increase the large animated
image cutoff from 1MB to 5MB.

In addition when pruning we will aggressively discard image sources.

Reviewed by ggaren

• platform/graphics/BitmapImage.cpp: (WebCore::BitmapImage::destroyDecodedData):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/BitmapImage.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2007-03-08  David Hyatt  <hyatt@apple.com>
+
+        Fix 2% performance regression on the PLT.  Increase the large animated
+        image cutoff from 1MB to 5MB.
+
+        In addition when pruning we will aggressively discard image sources.
+
+        Reviewed by ggaren
+
+        * platform/graphics/BitmapImage.cpp:
+        (WebCore::BitmapImage::destroyDecodedData):
+
 2007-03-08  Timothy Hatcher  <timothy@apple.com>
 

trunk/WebCore/platform/graphics/BitmapImage.cpp

@@ -38 +38 @@
 namespace WebCore {
 
-// Animated images >1MB are considered large enough that we'll only hang on to
+// Animated images >5MB are considered large enough that we'll only hang on to
 // one frame at a time.
-const unsigned cLargeAnimationCutoff = 1048576;
+const unsigned cLargeAnimationCutoff = 5242880;
 
 BitmapImage::BitmapImage(ImageObserver* observer)
@@ -93 +93 @@
         }
         
-        if (!incremental && frameCount() * frameSize > cLargeAnimationCutoff) {
+        if (!incremental) {
             // Reset the image source, since Image I/O has an underlying cache that it uses
             // while animating that it seems to never clear.