[patch] fix display of animated GIFs with net-libs/webkit-gtk-1.2.7

webkit-gtk-1.2.7-animated-gifs.patch (text/plain), 1.82 KB, created by David King on 2011-06-27 17:58:26 UTC

From 1c3ae92649aa57aad859484c7cbbdae69c73f6cd Mon Sep 17 00:00:00 2001
From: Gustavo Noronha Silva <[e-mail]>
Date: Tue, 8 Mar 2011 09:42:13 -0300
Subject: [PATCH] Fix bad merge that left trailing semicolons on a couple if statements

This was causing gifs to blink instead of animating. For more details
look here: https://bugs.webkit.org/show_bug.cgi?id=55830

Found and fixed by David Hill <[e-mail]>
---
 .../image-decoders/gif/GIFImageDecoder.cpp         |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/WebCore/platform/image-decoders/gif/GIFImageDecoder.cpp b/WebCore/platform/image-decoders/gif/GIFImageDecoder.cpp
index dc288fd..3bb2340 100644
--- a/WebCore/platform/image-decoders/gif/GIFImageDecoder.cpp
+++ b/WebCore/platform/image-decoders/gif/GIFImageDecoder.cpp
@@ -343,7 +343,7 @@ bool GIFImageDecoder::initFrameBuffer(unsigned frameIndex)
 
         if ((prevMethod == RGBA32Buffer::DisposeNotSpecified) || (prevMethod == RGBA32Buffer::DisposeKeep)) {
             // Preserve the last frame as the starting state for this frame.
-            if (!buffer->copyBitmapData(*prevBuffer));
+            if (!buffer->copyBitmapData(*prevBuffer))
                 return setFailed();
         } else {
             // We want to clear the previous frame to transparent, without
@@ -357,7 +357,7 @@ bool GIFImageDecoder::initFrameBuffer(unsigned frameIndex)
                     return setFailed();
             } else {
               // Copy the whole previous buffer, then clear just its frame.
-              if (!buffer->copyBitmapData(*prevBuffer));
+              if (!buffer->copyBitmapData(*prevBuffer))
                   return setFailed();
               for (int y = prevRect.y(); y < prevRect.bottom(); ++y) {
                   for (int x = prevRect.x(); x < prevRect.right(); ++x)
-- 
1.6.1