Skip to content

Conversation

@mukilan
Copy link
Member

@mukilan mukilan commented Nov 21, 2025

Currently, only the first Painter is used for all operations in the Compositor. This change modifies the compositor API and message handling to allow routing the operations to the correct Painter via a provided PainterId or WebViewId.

This change is to enable support for per-WebViews RenderingContexts.

Testing: This change shouldn't change behavior, so existing WPT tests should cover it.

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 21, 2025
Currently, only the first painter is used for all operations in the
compositor. This change modifies the compositor API and messages to
allow routing the operations to the correct painter.

This change is to enable support for WebViews with independent rendering
contexts.

Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com>
Co-authored-by: Martin Robinson <mrobinson@igalia.com>
@mukilan mukilan force-pushed the per-painter-compositor-operations branch from dc36f40 to 16c25ea Compare November 21, 2025 11:19
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 21, 2025
Signed-off-by: Martin Robinson <mrobinson@igalia.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 21, 2025
@mrobinson mrobinson enabled auto-merge November 21, 2025 13:10
@mrobinson mrobinson added this pull request to the merge queue Nov 21, 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 Nov 21, 2025
Merged via the queue into servo:main with commit 90595e3 Nov 21, 2025
35 checks passed
@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 Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants