Skip to content

compositing: Move WebRender initialization to IOCompositor creation#40101

Merged
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:initialize-webrender-inside-compositor
Oct 23, 2025
Merged

compositing: Move WebRender initialization to IOCompositor creation#40101
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:initialize-webrender-inside-compositor

Conversation

@mrobinson
Copy link
Copy Markdown
Member

In order to support multiple WebRender instances in the renderer, we
need the ability for the renderer itself to manage their creation and
destruction. This change moves the initialization of the (currently)
single WebRender instances to the renderer initialization.

Testing: This should not change behavior, so is covered by existing tests.

In order to support multiple WebRender instances in the renderer, we
need the ability for the renderer itself to manage their creation and
destruction. This change moves the initialization of the (currently)
single WebRender instances to the renderer initialization.

Co-authored-by: Mukilan Thiyagarajan <[email protected]>
Signed-off-by: Martin Robinson <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 23, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 23, 2025
@mrobinson mrobinson added this pull request to the merge queue Oct 23, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 23, 2025
Merged via the queue into servo:main with commit 183f141 Oct 23, 2025
33 checks passed
@mrobinson mrobinson deleted the initialize-webrender-inside-compositor branch October 23, 2025 13:02
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants