http://bugzilla.opendarwin.org/show_bug.cgi?id=10779
REGRESSION: Animated GIF ignores frame intervals and loops infinitely
No test possible.
* platform/Image.cpp:
(WebCore::Image::shouldAnimate): Don't check that there is more than one frame,
maybe the rest just hasn't been loaded yet.
(WebCore::Image::startAnimation): Move the frame count check here - there is no need
to start animating before we get at least two frames.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@16417
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2006-09-17 Alexey Proskuryakov <ap@nypop.com>
+
+ Reviewed by Hyatt.
+
+ http://bugzilla.opendarwin.org/show_bug.cgi?id=10779
+ REGRESSION: Animated GIF ignores frame intervals and loops infinitely
+
+ No test possible.
+
+ * platform/Image.cpp:
+ (WebCore::Image::shouldAnimate): Don't check that there is more than one frame,
+ maybe the rest just hasn't been loaded yet.
+ (WebCore::Image::startAnimation): Move the frame count check here - there is no need
+ to start animating before we get at least two frames.
+
2006-09-17 Brady Eidson <beidson@apple.com>
Reviewed by Sarge
bool Image::shouldAnimate()
{
- return (m_animatingImageType && frameCount() > 1 && !m_animationFinished && m_animationObserver);
+ return (m_animatingImageType && !m_animationFinished && m_animationObserver);
}
void Image::startAnimation()
{
- if (m_frameTimer || !shouldAnimate())
+ if (m_frameTimer || !shouldAnimate() || frameCount() <= 1)
return;
m_frameTimer = new Timer<Image>(this, &Image::advanceAnimation);