Skip to content

Conversation

@mertalev
Copy link
Member

@mertalev mertalev commented Sep 16, 2025

Description

It's possible for thumbnails to fail to load on iOS when the background worker starts running in the background. The app can attempt to process / queue tasks on the GPU when it isn't available (it isn't foreground), so the thumbnails are considered failed even when the app is opened. The new Flutter update should fix this.

Disables custom_lint for now as it needs a lot of dependency changes to work with the new version, which then need wider changes in the codebase. It can be re-enabled later without blocking the Flutter upgrade.

How Has This Been Tested?

Tested the main timeline, asset viewer, album page and map view on an iOS release build and did not notice any anomalies. Due to the inconsistency in reproducing this scenario, I can't verify that the GPU issue is fixed.

@alextran1502
Copy link
Member

Iirc @bwees reported that video player was broken on this flutter version. Please correct me if I misremembering

@mertalev
Copy link
Member Author

Video playback works for me. The blocker I'm facing is that custom_lint isn't working in the CI. I think we might need to update it to 0.8.1, which then also means analyzer needs to be bumped to 8.0 and that means we need to update our isar_generator fork to upstream? @shenlong-tanwen could you help with that?

@bwees
Copy link
Member

bwees commented Sep 16, 2025

Iirc @bwees reported that video player was broken on this flutter version. Please correct me if I misremembering

This may have been related to running a mismatched flutter cli version and project sdk

@mertalev mertalev requested a review from bo0tzz as a code owner September 16, 2025 16:10
@alextran1502 alextran1502 merged commit 585b74f into main Sep 17, 2025
53 checks passed
@alextran1502 alextran1502 deleted the chore/deps-flutter-3.35.3 branch September 17, 2025 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants