RESOLVED FIXED 231653
[TextureMapper] large blur filter doesn't look smooth 
https://bugs.webkit.org/show_bug.cgi?id=231653
Summary [TextureMapper] large blur filter doesn't look smooth
Fujii Hironori
Reported 2021-10-12 19:02:56 PDT
[TextureMapper] large blur filter doesn't look smooth attachment#413757 [details] https://speakerdeck.com/trishagee/code-review-best-practice
Attachments
test case (452 bytes, text/html)
2022-02-21 15:56 PST, Fujii Hironori 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-10-19 19:03:20 PDT
<rdar://problem/84444307>
Smoley
Comment 2 2021-11-02 17:19:27 PDT
Thanks for filing, the attached test case and the linked page both look smooth for me on Safari Technology Preview 134. Are you seeing this issue there as well?
Fujii Hironori
Comment 3 2021-11-02 18:23:25 PDT
TextureMapper is used only by GTK, WPE, WinCairo and PlayStation ports.
Fujii Hironori
Comment 4 2022-02-21 15:56:44 PST
Created attachment 452789 [details] test case
Fujii Hironori
Comment 5 2022-06-26 23:09:27 PDT
FEGaussianBlurSoftwareApplier::boxBlurUnaccelerated is applying box-blur three times. https://github.com/WebKit/WebKit/blob/a713d4c4e774133040e273ce5c2ec5e8ebf85e02/Source/WebCore/platform/graphics/filters/software/FEGaussianBlurSoftwareApplier.cpp#L302-L328 This is suggested by the spec: https://drafts.fxtf.org/filter-effects/#feGaussianBlurElement > For larger values of "s" (s >= 2.0), an approximation can be > used: Three successive box-blurs build a piece-wise quadratic > convolution kernel, which approximates the Gaussian kernel to > within roughly 3%.
Akihiro Kiuchi
Comment 6 2023-09-01 00:30:52 PDT
I will tackle this issue under mentorship of Fujii-san.
Fujii Hironori
Comment 7 2023-09-06 20:33:30 PDT
All sub tasks have been resovled. No more remaining tasks. Closed.
Carlos Garcia Campos
Comment 8 2023-09-07 00:40:02 PDT
Could you add a link here to the PR fixing this issue?
Fujii Hironori
Comment 9 2023-09-07 00:48:43 PDT
https://github.com/WebKit/WebKit/pull/17419 fixed the pixelated problem. https://github.com/WebKit/WebKit/pull/17471 is a speed up Gaussian blur filtrer.
Carlos Garcia Campos
Comment 10 2023-09-07 03:00:45 PDT
Thanks, great job!
Note You need to log in before you can comment on or make changes to this bug.