Stop requiring a RemoteRenderingBackend for visibility propagation views

• Component: WebKit2
• Status: RESOLVED FIXED

[Chris Dumez]

Stop requiring a RemoteRenderingBackend for visibility propagation views. This is inconvenient because doing media playback in the GPUProcess does not require a RemoteRenderingBackend.
This also makes things difficult to exit the GPUProcess on memory pressure when unused.

[Chris Dumez]

Created attachment 425922 [details]
Patch

[Geoffrey Garen]

Comment on attachment 425922 [details]
Patch

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

r=me

> Source/WebKit/WebProcess/WebPage/WebPage.cpp:857
> +#if ENABLE(GPU_PROCESS)
> +    if (auto* gpuProcessConnection = WebProcess::singleton().existingGPUProcessConnection())
> +        gpuProcessConnection->createVisibilityPropagationContextForPage(*this);
> +#endif // ENABLE(GPU_PROCESS)
> +#endif // HAVE(VISIBILITY_PROPAGATION_VIEW)

Feels slightly better to call gpuProcessConnectionDidBecomeAvailable() here instead -- in case someone adds "things to do when I get a GPU Process" logic in the future.

[Chris Dumez]

Created attachment 425928 [details]
Patch

[EWS]

Committed r275921 (236484@main): <https://commits.webkit.org/236484@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/76619462>

[Chris Dumez]

Follow-up in <https://commits.webkit.org/r275928> to address crashes on the iOS bot.