Title: [121208] trunk/Source/WebCore
Revision
121208
Author
[email protected]
Date
2012-06-25 18:26:17 -0700 (Mon, 25 Jun 2012)

Log Message

Reduce memory footprint of BitmapImage Vectors.
https://bugs.webkit.org/show_bug.cgi?id=86563

Patch by Nick Carter <[email protected]> on 2012-06-25
Reviewed by Simon Fraser.

No new tests, affects template params only.

* platform/graphics/BitmapImage.h:
(BitmapImage): use an inline capacity of 1 for the image frame Vector
* platform/image-decoders/ImageDecoder.h:
(ImageDecoder): use an inline capacity of 1 for the image frame Vector

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121207 => 121208)


--- trunk/Source/WebCore/ChangeLog	2012-06-26 01:20:05 UTC (rev 121207)
+++ trunk/Source/WebCore/ChangeLog	2012-06-26 01:26:17 UTC (rev 121208)
@@ -1,3 +1,17 @@
+2012-06-25  Nick Carter  <[email protected]>
+
+        Reduce memory footprint of BitmapImage Vectors.
+        https://bugs.webkit.org/show_bug.cgi?id=86563
+
+        Reviewed by Simon Fraser.
+
+        No new tests, affects template params only.
+
+        * platform/graphics/BitmapImage.h:
+        (BitmapImage): use an inline capacity of 1 for the image frame Vector 
+        * platform/image-decoders/ImageDecoder.h:
+        (ImageDecoder): use an inline capacity of 1 for the image frame Vector
+
 2012-06-25  Kent Tamura  <[email protected]>
 
         Unreviewed, rolling out r121145.

Modified: trunk/Source/WebCore/platform/graphics/BitmapImage.h (121207 => 121208)


--- trunk/Source/WebCore/platform/graphics/BitmapImage.h	2012-06-26 01:20:05 UTC (rev 121207)
+++ trunk/Source/WebCore/platform/graphics/BitmapImage.h	2012-06-26 01:26:17 UTC (rev 121208)
@@ -272,8 +272,8 @@
     mutable IntSize m_sizeRespectingOrientation;
     
     size_t m_currentFrame; // The index of the current frame of animation.
-    Vector<FrameData> m_frames; // An array of the cached frames of the animation. We have to ref frames to pin them in the cache.
-    
+    Vector<FrameData, 1> m_frames; // An array of the cached frames of the animation. We have to ref frames to pin them in the cache.
+
     Timer<BitmapImage>* m_frameTimer;
     int m_repetitionCount; // How many total animation loops we should do.  This will be cAnimationNone if this image type is incapable of animation.
     RepetitionCountStatus m_repetitionCountStatus;

Modified: trunk/Source/WebCore/platform/image-decoders/ImageDecoder.h (121207 => 121208)


--- trunk/Source/WebCore/platform/image-decoders/ImageDecoder.h	2012-06-26 01:20:05 UTC (rev 121207)
+++ trunk/Source/WebCore/platform/image-decoders/ImageDecoder.h	2012-06-26 01:26:17 UTC (rev 121208)
@@ -362,7 +362,7 @@
         int scaledY(int origY, int searchStart = 0);
 
         RefPtr<SharedBuffer> m_data; // The encoded data.
-        Vector<ImageFrame> m_frameBufferCache;
+        Vector<ImageFrame, 1> m_frameBufferCache;
         // FIXME: Do we need m_colorProfile any more, for any port?
         ColorProfile m_colorProfile;
         bool m_scaled;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to