14413 2007-06-26 09:47:45 -0700 GIF animation throttling is different from MSIE 2010-03-13 10:11:25 -0800 1 1 1 Unclassified WebKit Images 523.x (Safari 3) Mac OS X 10.4 RESOLVED FIXED https://bugzilla.mozilla.org/skins/custom/images/mozchomp.gif P2 Normal --- 1 tor ap ap gavin.sharp hyatt mrowe oldest_to_newest 6193 0 tor 2007-06-26 09:47:45 -0700 IE and Opera have a minimum delay time between GIF frames, empirically found to be 100ms. Mozilla also has code for this, though it was broken in FF2. Safari does not appear to have a frame rate throttle, or if it does, it appears to be set below the other browsers. Test URL appears to be running unthrottled in Safari 3.0.2 (windows) and webkit nightly (os-x). 6194 1 ap 2007-06-26 10:01:13 -0700 This is from WebCore code: // We follow Firefox's behavior and use a duration of 100 ms for any frames that specify // a duration of <= 10 ms. See gfxImageFrame::GetTimeout in Gecko or Radar 4051389 for more. if (duration <= 0.010) return 0.100; 6202 2 ap 2007-06-26 10:50:26 -0700 http://nypop.com/~ap/webkit/mozchomp/mozchomp.html According to this test, the behavior is: IE 7/Win: <=50ms -> 100ms >50ms -> as specified Opera 9.2/Mac: <100ms -> 100ms >=100ms -> as specified Safari 2&3, Firefox 2: <=10ms -> 100ms >10ms -> as specified 6204 3 15252 ap 2007-06-26 11:23:30 -0700 Created attachment 15252 proposed fix Comparing with 0.051 is needed because of floating point error, and is safe because GIF specifies timeouts in tens of milliseconds. Please note that the test case still renders incorrectly on my machine - 100ms and 110ms images animate synchronously! This appears to be some unrelated timer-related issue: on a simpler page with just these two images left, the same images are animated correctly. 6160 4 15252 ggaren 2007-06-26 14:17:27 -0700 Comment on attachment 15252 proposed fix r=me Nice testing! 6157 5 ggaren 2007-06-26 14:18:05 -0700 CC'ing Dave in case he disagrees. I believe he wrote the original code. 6158 6 ggaren 2007-06-26 14:18:32 -0700 Do we need a new bug about the timer-related issue you've seen? 6078 7 ap 2007-06-27 10:23:27 -0700 Committed revision 23821; filed bug 14432. 5859 8 ap 2007-06-30 03:20:27 -0700 See also: <https://bugzilla.mozilla.org/show_bug.cgi?id=386269#c4> (it's not a given that Mozilla will follow IE behavior). 84018 9 pkasting 2008-06-20 16:54:02 -0700 Firefox 3 shipped with the same behavior as past Firefox releases: <= 10 ms -> 100 ms. As far as I know, they are the only browser to do this now, and I filed https://bugzilla.mozilla.org/show_bug.cgi?id=440882 about matching the other engines. That said, I think Gecko's behavior, and WebKit's old behavior, is superior to WebKit's current behavior in the abstract, and if Gecko refuses to change, WebKit should seriously consider reverting this fix. 199368 10 ap 2010-03-13 10:11:25 -0800 (In reply to comment #9) > That said, I think Gecko's behavior, and WebKit's old behavior, > is superior to WebKit's current behavior in the abstract, and if Gecko refuses > to change, WebKit should seriously consider reverting this fix. We are seriously considering that, see bug 36082. We don't have a consensus yet. 15252 2007-06-26 11:23:30 -0700 2007-06-26 14:17:27 -0700 proposed fix 14413r1_patch.txt text/plain 1556 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyMzc5NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDctMDYtMjYgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJr aXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NDEzCisgICAgICAgIEdJ RiBhbmltYXRpb24gdGhyb3R0bGluZyBpcyBkaWZmZXJlbnQgZnJvbSBNU0lFCisKKyAgICAgICAg KiBwbGF0Zm9ybS9ncmFwaGljcy9jZy9JbWFnZVNvdXJjZUNHLmNwcDoKKyAgICAgICAgKFdlYkNv cmU6OkltYWdlU291cmNlOjpmcmFtZUR1cmF0aW9uQXRJbmRleCk6CisgICAgICAgIENoYW5nZSB0 aGUgY3V0LW9mZiB2YWx1ZSB0byA1MG1zIHRvIG1hdGNoIE1TSUUuCisKIDIwMDctMDYtMjYgIFRp bW90aHkgSGF0Y2hlciAgPHRpbW90aHlAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5 IE1hcmsgUm93ZS4KSW5kZXg6IFdlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvY2cvSW1hZ2VTb3Vy Y2VDRy5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9jZy9JbWFn ZVNvdXJjZUNHLmNwcAkocmV2aXNpb24gMjM3OTQpCisrKyBXZWJDb3JlL3BsYXRmb3JtL2dyYXBo aWNzL2NnL0ltYWdlU291cmNlQ0cuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNzcsOSArMTc3LDkg QEAgZmxvYXQgSW1hZ2VTb3VyY2U6OmZyYW1lRHVyYXRpb25BdEluZGV4KAogICAgIH0KIAogICAg IC8vIE1hbnkgYW5ub3lpbmcgYWRzIHNwZWNpZnkgYSAwIGR1cmF0aW9uIHRvIG1ha2UgYW4gaW1h Z2UgZmxhc2ggYXMgcXVpY2tseSBhcyBwb3NzaWJsZS4KLSAgICAvLyBXZSBmb2xsb3cgRmlyZWZv eCdzIGJlaGF2aW9yIGFuZCB1c2UgYSBkdXJhdGlvbiBvZiAxMDAgbXMgZm9yIGFueSBmcmFtZXMg dGhhdCBzcGVjaWZ5Ci0gICAgLy8gYSBkdXJhdGlvbiBvZiA8PSAxMCBtcy4gU2VlIGdmeEltYWdl RnJhbWU6OkdldFRpbWVvdXQgaW4gR2Vja28gb3IgUmFkYXIgNDA1MTM4OSBmb3IgbW9yZS4KLSAg ICBpZiAoZHVyYXRpb24gPD0gMC4wMTApCisgICAgLy8gV2UgZm9sbG93IFdpbklFJ3MgYmVoYXZp b3IgYW5kIHVzZSBhIGR1cmF0aW9uIG9mIDEwMCBtcyBmb3IgYW55IGZyYW1lcyB0aGF0IHNwZWNp ZnkKKyAgICAvLyBhIGR1cmF0aW9uIG9mIDw9IDUwIG1zLiBTZWUgPGh0dHA6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTE0NDEzPiBvciBSYWRhciA0MDUxMzg5IGZvciBtb3JlLgor ICAgIGlmIChkdXJhdGlvbiA8IDAuMDUxKQogICAgICAgICByZXR1cm4gMC4xMDA7CiAgICAgcmV0 dXJuIGR1cmF0aW9uOwogfQo=