Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DRAFT] Send dispose command for CompositionTarget as an OOB batch #18119

Merged
merged 6 commits into from
Feb 26, 2025

Conversation

kekekeks
Copy link
Member

@kekekeks kekekeks commented Feb 4, 2025

When a window or popup gets closed we need to make sure that any associated render thread operation have been completed. To do so, we are currently sending a composition batch and waiting for it to complete. However triggering such commit outside of our normal commit scheduling causes some kind of "tearing", as described in #17784

This PR sends a dispose command in a separate small "out-of-baund" batch.

@kekekeks kekekeks added the backport-candidate-11.2.x Consider this PR for backporting to 11.2 branch label Feb 4, 2025
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.3.999-cibuild0054770-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@danwalmsley
Copy link
Member

@kekekeks unfortunately this still causes a render pass without a layout pass when the popup closes.

image

@kekekeks kekekeks changed the title Send dispose command for CompositionTarget as an OOB batch [DRAFT] Send dispose command for CompositionTarget as an OOB batch Feb 15, 2025
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.3.999-cibuild0055149-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

Copy link
Member

@danwalmsley danwalmsley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tested and confirmed fix, MacOS and Windows.

@jmacato jmacato added this pull request to the merge queue Feb 26, 2025
Merged via the queue into master with commit abd7a88 Feb 26, 2025
11 checks passed
@jmacato jmacato deleted the feature/oob-composition-target-dispose branch February 26, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-candidate-11.2.x Consider this PR for backporting to 11.2 branch bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants