Changeset 58744 in webkit
- Timestamp:
- May 4, 2010 2:15:05 AM (14 years ago)
- Location:
- trunk/WebCore
- Files:
-
- 1 added
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/WebCore/ChangeLog
r58743 r58744 1 2010-05-04 Tucker Jay <jay.tucker@nokia.com> 2 3 Reviewed by Holger Freyther. 4 5 Animated GIF images does not animate 10x as expected by default. 6 https://bugs.webkit.org/show_bug.cgi?id=36818 7 8 Added test case to existing manual test to test the 9 fixed functionality. 10 11 * manual-tests/qt/qt-10loop-anim.gif: Added. 12 * manual-tests/qt/qt-gif-test.html: 13 * platform/graphics/qt/ImageDecoderQt.cpp: 14 (WebCore::ImageDecoderQt::repetitionCount): 15 1 16 2010-05-04 Dirk Schulze <krit@webkit.org> 2 17 -
trunk/WebCore/manual-tests/qt/qt-gif-test.html
r55774 r58744 3 3 <div> 4 4 <img src="qt-anim.gif"> 5 <p>This should animate </p>5 <p>This should animate.</p> 6 6 </div> 7 7 <div> 8 8 <img src="qt-noanim.gif"> 9 <p>This should not animate and you should see a cross</p> 9 <p>This should not animate and you should see a cross.</p> 10 </div 11 <div> 12 <img src="qt-10loop-anim.gif"> 13 <p>This animation should loop 10 times.</p> 10 14 </div 11 15 </body> -
trunk/WebCore/platform/graphics/qt/ImageDecoderQt.cpp
r57979 r58744 124 124 // -1 | 0 | infinite animation 125 125 // 0 | cAnimationLoopOnce | show every frame once 126 // n | n | no idea if that is supported126 // n | n+1 | Qt returns the # of iterations - 1 127 127 // n/a | cAnimationNone | show only the first frame 128 128 if (m_repetitionCount == -1) … … 130 130 else if (m_repetitionCount == 0) 131 131 m_repetitionCount = cAnimationLoopOnce; 132 else 133 ++m_repetitionCount; 132 134 } 133 135
Note: See TracChangeset
for help on using the changeset viewer.