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

Fix bug where ImageGenerator factories with the same priority get dropped in release mode #29254

Merged
merged 1 commit into from
Oct 20, 2021

Conversation

bdero
Copy link
Member

@bdero bdero commented Oct 20, 2021

Fixes flutter/flutter#91421.

This was a silly one. I naively assumed that fml::tracing::TraceNonce() was intended for global use, but in retrospect it's quite obvious that it's only intended for trace ordering. In release mode it's preprocessed out and only returns 0.

@bdero bdero requested a review from zanderso October 20, 2021 12:18
@google-cla google-cla bot added the cla: yes label Oct 20, 2021
@flutter flutter deleted a comment from flutter-dashboard bot Oct 20, 2021
@bdero bdero removed the needs tests label Oct 20, 2021
@bdero bdero force-pushed the bdero/fix-91421 branch 2 times, most recently from cd51690 to 3c1e736 Compare October 20, 2021 12:45
@bdero bdero force-pushed the bdero/fix-91421 branch 3 times, most recently from 44740c9 to 9c73ce4 Compare October 20, 2021 20:23
@bdero bdero requested a review from zanderso October 20, 2021 21:33
@zakton5
Copy link

zakton5 commented Nov 1, 2021

@bdero Any chance this could make it into a hotfix? Images currently aren't working in my app properly because of this. If not, is there an approximate ETA for the stable channel?

@bdero
Copy link
Member Author

bdero commented Nov 1, 2021

@zakton5 I went ahead and labeled this as a requested cherrypick for 2.5 stable. I agree this would be good to include in the next hotfix (the HEIC regression was introduced here: #25987).

yx-mike added a commit to yx-mike/engine that referenced this pull request May 18, 2022
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.

HEIC images are not rendered in release mode, throws Exception: Invalid Image Data
3 participants