250481 2023-01-11 15:15:22 -0800 Improve comparison of intervals in SVGSMILElement::resolveFirstInterval 2025-08-25 07:56:25 -0700 1 1 1 Unclassified WebKit SVG WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 ahmad.saleem792 ahmad.saleem792 karlcow sabouhallawa webkit-bug-importer zimmermann oldest_to_newest 1924907 0 ahmad.saleem792 2023-01-11 15:15:22 -0800 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! 1924979 1 karlcow 2023-01-11 20:59:10 -0800 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. 1925029 2 ahmad.saleem792 2023-01-12 04:21:26 -0800 (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 1926430 3 webkit-bug-importer 2023-01-18 15:16:14 -0800 <rdar://problem/104399900> 2051960 4 ahmad.saleem792 2024-08-11 01:15:18 -0700 SVN repo is gone - so alternative link - https://github.com/chromium/chromium/commit/5b93bdd34c17dae682ce14c45125abc603310b16 2138486 5 ahmad.saleem792 2025-08-23 12:32:21 -0700 Pull request: https://github.com/WebKit/WebKit/pull/49818 2138608 6 ews-feeder 2025-08-25 07:56:23 -0700 Committed 299110@main (661903c79498): <https://commits.webkit.org/299110@main> Reviewed commits have been landed. Closing PR #49818 and removing active labels.