Title: [132666] trunk/Source/WebCore
Revision
132666
Author
[email protected]
Date
2012-10-26 08:58:42 -0700 (Fri, 26 Oct 2012)

Log Message

Clear m_orientation in FrameData::clear() for skia
https://bugs.webkit.org/show_bug.cgi?id=100456

Reviewed by Stephen White.

clear() is only called by the FrameData destructor and for multi-image
images in BitmapImage::destroyDecodedData(). Multi-frame images don't
have exif data, so this patch should have no effect in practice. It
makes the skia code match the CG code in BitmapImageCG.cpp though.

* platform/graphics/skia/ImageSkia.cpp:
(WebCore::FrameData::clear):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (132665 => 132666)


--- trunk/Source/WebCore/ChangeLog	2012-10-26 15:57:08 UTC (rev 132665)
+++ trunk/Source/WebCore/ChangeLog	2012-10-26 15:58:42 UTC (rev 132666)
@@ -1,3 +1,18 @@
+2012-10-26  Nico Weber  <[email protected]>
+
+        Clear m_orientation in FrameData::clear() for skia
+        https://bugs.webkit.org/show_bug.cgi?id=100456
+
+        Reviewed by Stephen White.
+
+        clear() is only called by the FrameData destructor and for multi-image
+        images in BitmapImage::destroyDecodedData(). Multi-frame images don't
+        have exif data, so this patch should have no effect in practice. It
+        makes the skia code match the CG code in BitmapImageCG.cpp though.
+
+        * platform/graphics/skia/ImageSkia.cpp:
+        (WebCore::FrameData::clear):
+
 2012-10-26  Florin Malita  <[email protected]>
 
         Crash on loading SVG filter resource on HTML element

Modified: trunk/Source/WebCore/platform/graphics/skia/ImageSkia.cpp (132665 => 132666)


--- trunk/Source/WebCore/platform/graphics/skia/ImageSkia.cpp	2012-10-26 15:57:08 UTC (rev 132665)
+++ trunk/Source/WebCore/platform/graphics/skia/ImageSkia.cpp	2012-10-26 15:58:42 UTC (rev 132666)
@@ -449,6 +449,8 @@
     if (clearMetadata)
         m_haveMetadata = false;
 
+    m_orientation = DefaultImageOrientation;
+
     if (m_frame) {
         // ImageSource::createFrameAtIndex() allocated |m_frame| and passed
         // ownership to BitmapImage; we must delete it here.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to