Bug 250313
| Summary: | Simplify SMILTime operators and use NaN/Inf to represent 'unresolved/indefinite' SMILTimes | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Ahmad Saleem <ahmad.saleem792> |
| Status: | NEW | ||
| Severity: | Normal | CC: | sabouhallawa, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While going through Blink's commits, I came across another nice clean-up, it might be good to have similar in WebKit IMO.
WebKit Source - https://github.com/WebKit/WebKit/blob/bc2b6ffeef9e0b0b554d392cb5e914a61b69b42e/Source/WebCore/svg/animation/SMILTime.cpp#L37
Blink Commit - https://chromium.googlesource.com/chromium/blink/+/f3d854b85d8204d3f2ab2bda895b464f884e090f
Just wanted to raise it so we can track it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
Pre-requisite - https://src.chromium.org/viewvc/blink?view=revision&revision=180823
Radar WebKit Bug Importer
<rdar://problem/104295073>
Ahmad Saleem
(In reply to Ahmad Saleem from comment #1)
> Pre-requisite -
> https://src.chromium.org/viewvc/blink?view=revision&revision=180823
SVN is gone - so replacement link https://chromium.googlesource.com/chromium/blink/+/7fe1ac80f6b2e96a4cb675776d147598e4e1f4e6
Ahmad Saleem
Pull request: https://github.com/WebKit/WebKit/pull/32010