Bug 35065 – Delay between page loading and animated GIF playing

• Status: RESOLVED
• Resolution: FIXED
• Component: Images

[Mark Rowe (bdash)]

When doing an uncached load of <http://i.imgur.com/Zgc5p.gif> there is a delay of around 5 seconds after the image finishes loading before the animation starts running.

<rdar://problem/7109548>

[Mark Rowe (bdash)]

Created attachment 48955 [details]
Patch v1

This appears to address the issue.

[Mark Rowe (bdash)]

CCing Mr Peter Kasting as he did a lot of work on this code back in the day.

[Mark Rowe (bdash)]

Fixed in r54919.

[Peter Kasting]

The old code was definitely way wrong.  Thanks for fixing!

The new patch has a small bug: the first time through, when the desired frame start time is zero, we should set the desired start time immediately.  Otherwise the GIF start lags by the amount of time needed to decode the second frame.  This is minor, but can make a difference on sites like YTMND.  I'll write a patch for this.

[Peter Kasting]

Comment on attachment 48955 [details]
Patch v1

Clearing flags on this patch, as it has landed.

[Mark Rowe (bdash)]

(In reply to comment #4)
> The old code was definitely way wrong.  Thanks for fixing!
> 
> The new patch has a small bug: the first time through, when the desired frame
> start time is zero, we should set the desired start time immediately. 
> Otherwise the GIF start lags by the amount of time needed to decode the second
> frame.  This is minor, but can make a difference on sites like YTMND.  I'll
> write a patch for this.

Please file a new bug.  That’s clearly a different issue than what this bug was about, so repurposing it for that doesn’t seem sensible.