Hi Team, While going through Blink's commit, came across another potential fix: Commit - https://src.chromium.org/viewvc/blink?revision=175122&view=revision WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/svg/animation/SVGSMILElement.cpp#863 etc. Just wanted to raise to see if we can fix this. Thanks!
The prose of the Blink commit. ============ Improve comparison of intervals in SVGSMILElement::resolveFirstInterval For open-ended intervals such as [3) - i.e. begin=3s and end/duration/etc unspecified - every call to resolveFirstInterval() would think that a new interval had been created, and notify/require a reschedule of the animations. Add a new type SMILInterval and a suitable operator for that, and then use it to compare the raw values (i.e. if both endpoints are exacly the same.) BUG=377329 =========== https://bugs.chromium.org/p/chromium/issues/detail?id=377329 There is a testcase in there.
(In reply to Karl Dubost from comment #1) > The prose of the Blink commit. > > ============ > Improve comparison of intervals in SVGSMILElement::resolveFirstInterval > > For open-ended intervals such as [3) - i.e. begin=3s and end/duration/etc > unspecified - every call to resolveFirstInterval() would think that a new > interval had been created, and notify/require a reschedule of the > animations. > Add a new type SMILInterval and a suitable operator for that, and then > use it to compare the raw values (i.e. if both endpoints are exacly the > same.) > > BUG=377329 > =========== > https://bugs.chromium.org/p/chromium/issues/detail?id=377329 > > > There is a testcase in there. The test need: Add a printf in SVGImageChromeClient::invalidateContentsAndRootView
<rdar://problem/104399900>