Unreviewed, rolling out r101302.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Nov 2011 09:23:28 +0000 (09:23 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 29 Nov 2011 09:23:28 +0000 (09:23 +0000)
http://trac.webkit.org/changeset/101302
https://bugs.webkit.org/show_bug.cgi?id=73293

massive crashes on gtk/win bots - m_bytes assertion fires
(Requested by WildFox on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-11-29

* platform/image-decoders/ImageDecoder.cpp:
(WebCore::ImageFrame::setSize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@101340 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/image-decoders/ImageDecoder.cpp

index eef9dfa..e11741c 100644 (file)
@@ -1,3 +1,15 @@
+2011-11-29  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r101302.
+        http://trac.webkit.org/changeset/101302
+        https://bugs.webkit.org/show_bug.cgi?id=73293
+
+        massive crashes on gtk/win bots - m_bytes assertion fires
+        (Requested by WildFox on #webkit).
+
+        * platform/image-decoders/ImageDecoder.cpp:
+        (WebCore::ImageFrame::setSize):
+
 2011-11-29  Sean Wang  <Xuewen.Wang@torchmobile.com.cn>
 
         Upstream BlackBerry porting of platform/image-decoders
index f025fbb..ca6369e 100644 (file)
@@ -189,14 +189,14 @@ bool ImageFrame::copyBitmapData(const ImageFrame& other)
 
 bool ImageFrame::setSize(int newWidth, int newHeight)
 {
-    ASSERT(!m_bytes);
-    size_t backingStoreSize = newWidth * newHeight;
-    if (!m_backingStore.tryReserveCapacity(backingStoreSize))
-        return false;
+    // NOTE: This has no way to check for allocation failure if the requested
+    // size was too big...
+    m_backingStore.resize(newWidth * newHeight);
     m_bytes = m_backingStore.data();
     m_size = IntSize(newWidth, newHeight);
 
     zeroFillPixelData();
+
     return true;
 }