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

If the application says there is a new texture but does not provide one, reuse the last texture.#17437

Merged
chinmaygarde merged 1 commit intoflutter:masterfrom
chinmaygarde:retain_old_external_texture
Apr 1, 2020
Merged

If the application says there is a new texture but does not provide one, reuse the last texture.#17437
chinmaygarde merged 1 commit intoflutter:masterfrom
chinmaygarde:retain_old_external_texture

Conversation

@chinmaygarde
Copy link
Contributor

This matches the behavior of the OpenGL backend.

…ne, reuse the last texture.

This matches the behavior of the OpenGL backend.
Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

RSLGTM

@chinmaygarde
Copy link
Contributor Author

This may happen with certain plugins that don't anticipate the asynchronous nature of the texture resolution say there is a texture frame ready but don't provide one when asked. This issue only happens with one known plugin and was not caught when I was testing all known applications with Metal earlier because the device I was using was super old and the time it took for Flutter to ask for the texture was long enough that a texture was always ready. The same may fail on newer/faster devices. I have tested all applications on newer devices just to be safe.

@chinmaygarde chinmaygarde merged commit 0911a08 into flutter:master Apr 1, 2020
@chinmaygarde chinmaygarde deleted the retain_old_external_texture branch April 1, 2020 05:01
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 1, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 1, 2020
goderbauer pushed a commit to goderbauer/engine that referenced this pull request Apr 16, 2020
…ne, reuse the last texture. (flutter#17437)

This matches the behavior of the OpenGL backend.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants