Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@flar
Copy link
Contributor

@flar flar commented Sep 13, 2023

@flar flar requested a review from jonahwilliams September 13, 2023 22:25
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@flar flar marked this pull request as draft September 14, 2023 19:41
@flar
Copy link
Contributor Author

flar commented Sep 14, 2023

This PR will become obsolete (if/)when #43759 is reverted.

@jonahwilliams
Copy link
Contributor

We should still land this if we can, as the preroll flag is used elsewhere too.

@chinmaygarde chinmaygarde changed the title Set impeller_enabled flag consistently in Preroll and Paint [Impeller] Set impeller_enabled flag consistently in Preroll and Paint. Sep 18, 2023
@jonahwilliams jonahwilliams marked this pull request as ready for review September 20, 2023 17:09
@jonahwilliams jonahwilliams marked this pull request as draft September 20, 2023 18:42
@jonahwilliams
Copy link
Contributor

nvm, you were right. I backed the change out manually and the flag could be removed entirely

@flar
Copy link
Contributor Author

flar commented Sep 26, 2023

Closing as obsolete

@flar flar closed this Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

[Impeller] LayerTree::Preroll and ::Paint use different criteria for impeller_enabled flag.

3 participants