<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:sabouhallawa@apple.com" title="Said Abou-Hallawa <sabouhallawa@apple.com>"> <span class="fn">Said Abou-Hallawa</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Animated GIFs fail to play in multi-column layout"
href="https://bugs.webkit.org/show_bug.cgi?id=167901">bug 167901</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>bfulgham@webkit.org, hyatt@apple.com, sabouhallawa@apple.com, zalan@apple.com
</td>
</tr>
<tr>
<td style="text-align:right;">Component</td>
<td>Images
</td>
<td>Layout and Rendering
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Animated GIFs fail to play in multi-column layout"
href="https://bugs.webkit.org/show_bug.cgi?id=167901#c2">Comment # 2</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Animated GIFs fail to play in multi-column layout"
href="https://bugs.webkit.org/show_bug.cgi?id=167901">bug 167901</a>
from <span class="vcard"><a class="email" href="mailto:sabouhallawa@apple.com" title="Said Abou-Hallawa <sabouhallawa@apple.com>"> <span class="fn">Said Abou-Hallawa</span></a>
</span></b>
<pre>This is not an image drawing or animation issue. It is a rendering issue. I think it is related to column layout.
In the test case <a href="https://jsfiddle.net/sdxaxuv5/">https://jsfiddle.net/sdxaxuv5/</a>, RenderElement::newImageAnimationFrameAvailable(), which is called via BitmapImage::advanceAnimation(, succeeds to call RenderObject::imageChanged() for the first four <img> elements in the column <div> element. But it fails to call it for the last two <img> elements. This happens because shouldRepaintForImageAnimation() returns true for the first four images but it returns false for the last two images.
Because the image rect is not marked for painting, the last two images do not animate. Fixing the repaint issue should force the image to draw the next image frame.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>