[GPU Process] [Filters] Enable Filter rendering in GPU Process

[Said Abou-Hallawa]

This will move applying the CSSFilters and the CSSFilterImageValue to GPUProcess. For now if the CSSFilter includes a referenced SVGFilter the whole CSSFilter will be applied in WebProcess.

[Said Abou-Hallawa]

Created attachment 446317 [details]
Patch

[Said Abou-Hallawa]

I will make this bug enables CSSFilter and SVGFilter in GPUProcess.

[Said Abou-Hallawa]

*** Bug 234027 has been marked as a duplicate of this bug. ***

[Said Abou-Hallawa]

Created attachment 446597 [details]
Patch

[Cameron McCormack (:heycam)]

Does bug 234099 (blocking this one) need to be done before this one?

[Cameron McCormack (:heycam)]

Comment on attachment 446597 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=446597&action=review

> Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp:157
> +    auto hostWindow = renderer.view().frameView().root() ? renderer.view().frameView().root()->hostWindow() : nullptr;

(I see this fragment so often that I wonder if createImageBuffer functions should take the renderer as an argument instead of the host window.)

[Darin Adler]

Comment on attachment 446597 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=446597&action=review

>> Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp:157
>> +    auto hostWindow = renderer.view().frameView().root() ? renderer.view().frameView().root()->hostWindow() : nullptr;
> 
> (I see this fragment so often that I wonder if createImageBuffer functions should take the renderer as an argument instead of the host window.)

Or just a hostWindow helper function so there’s not a separate line of code every time.

[Said Abou-Hallawa]

Created attachment 446656 [details]
Patch

[EWS]

Committed r286835 (245068@main): <https://commits.webkit.org/245068@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 446656 [details].

[Radar WebKit Bug Importer]

<rdar://problem/86310865>