89687 2012-06-21 13:17:35 -0700 Reloading a cached SVG image does not reset animations 2022-07-15 15:17:38 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID P2 Normal --- 89846 1 pdr pdr bfulgham zimmermann oldest_to_newest 654821 0 pdr 2012-06-21 13:17:35 -0700 If a user refreshes a page, the animation continues from the exact state it was in just prior to the refresh. There is a reproduction on this page: http://socializepalmsprings.com/archivereference/test-for-chromium/svg-as-img.html Original bug: crbug.com/132870 655447 1 zimmermann 2012-06-22 07:33:48 -0700 From SVGImage.cpp: void SVGImage::drawSVGToImageBuffer(ImageBuffer* buffer, const IntSize& size, float zoom, float scale, ShouldClearBuffer shouldClear) { // FIXME: This doesn't work correctly with animations. If an image contains animations, that say run for 2 seconds, // and we currently have one <img> that displays us. If we open another document referencing the same SVGImage it // will display the document at a time where animations already ran - even though it has its own ImageBuffer. // We currently don't implement SVGSVGElement::setCurrentTime, and can NOT go back in time, once animations started. // There's no way to fix this besides avoiding style/attribute mutations from SVGAnimationElement. Easy fixable today, as we have setCurrentTime support which is well-tested. I'm glad this can be resolved now. 656366 2 pdr 2012-06-24 20:51:48 -0700 (In reply to comment #1) > Easy fixable today, as we have setCurrentTime support which is well-tested. I'm glad this can be resolved now. If only it were so simple sometimes :P https://bugs.webkit.org/show_bug.cgi?id=89846 656446 3 zimmermann 2012-06-25 02:37:33 -0700 (In reply to comment #2) > (In reply to comment #1) > > Easy fixable today, as we have setCurrentTime support which is well-tested. I'm glad this can be resolved now. > > If only it were so simple sometimes :P > https://bugs.webkit.org/show_bug.cgi?id=89846 Heh, I didn't claim it's bug free, but at least setCurrentTime exists now... When I wrote the comment in SVGImage.cpp, it seemed like light-years away for us to support it, glad it changed :-) 1884286 4 bfulgham 2022-07-15 15:17:38 -0700 The relevant test site no longer exists. We can't take further action on this bug report.