Bug 262046
| Summary: | Treat huge 'repeatCount' values as invalid | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | graouts, sabouhallawa, simon.fraser, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar, WPTImpact |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://wpt.live/svg/animations/repeatcount-numeric-limit.tentative.svg | ||
Ahmad Saleem
Hi Team,
Came across another failing WPT test and also Blink merge:
Blink Commit: https://chromium.googlesource.com/chromium/src/+/8fcb33ec493a697c5973fed3fa3afaf77bf626db
WebKit Source Code: https://searchfox.org/wubkat/source/Source/WebCore/svg/animation/SVGSMILElement.cpp#722
^ Just adding '&& std::isfinite(result)'.
Test Case: In URL (Timeout in Safari / WebKit ToT)
Local build confirmed that it make us pass the test case.
Just raising so can fix it.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
EWS
Committed 268420@main (67ce0e567581): <https://commits.webkit.org/268420@main>
Reviewed commits have been landed. Closing PR #18165 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/116023138>