Bug 258229 - Don't cap the FEGaussian max kernel size
Summary: Don't cap the FEGaussian max kernel size
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: https://jsfiddle.net/cdfLgo7b/2/
Keywords: BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2023-06-16 17:21 PDT by Ahmad Saleem
Modified: 2023-06-23 17:22 PDT (History)
6 users (show)

See Also:


Attachments
All browsers result in test case (350.25 KB, image/png)
2023-06-16 17:24 PDT, Ahmad Saleem
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-06-16 17:21:31 PDT
Hi Team,

While going through WebKit's commit, I came across another test case where all browsers are different and by merging Blink commit, we start doing bit better (IMO?).

Blink Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=186886

WebKit SOurce: https://searchfox.org/wubkat/source/Source/WebCore/platform/graphics/filters/FEGaussianBlur.cpp#87

^ Remove Line 87 and change Line 92 as below:

return clampTo<int>(size);

_______

I think getting some input from Simon, Said and Nikolas would be good.

Thanks!
Comment 1 Ahmad Saleem 2023-06-16 17:24:32 PDT
Created attachment 466720 [details]
All browsers result in test case
Comment 2 Radar WebKit Bug Importer 2023-06-23 17:22:14 PDT
<rdar://problem/111241971>