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