Crash bij verlaten videodia: setState tijdens gelockte widgettree (VideoPlayheadBus.clearFor) #1114

New issue

Closed

opened 2026-08-02 08:33:47 +00:00 by brenno · 0 comments

brenno commented

2026-08-02 08:33:47 +00:00

Owner

Symptoom

Bij het weg-navigeren van een dia met (afspelende) video gooit Flutter tijdens het tekenen van het frame:

setState() or markNeedsBuild() called when widget tree was locked.
This ValueListenableBuilder<VideoPlayhead?> widget cannot be marked as needing to build
because the framework is locked.

De crash komt uit VideoPlayheadBus.clearFor → _VideoPreviewState.dispose tijdens BuildOwner.finalizeTree.

Oorzaak

VideoPlayheadBus.clearFor (lib/widgets/slides/video_playhead_bus.dart) zet current.value = null. Dat gebeurt vanuit dispose() van _VideoPreviewState én _VideoEmbedPreviewState (lib/widgets/slides/previews/media_previews_video.dart), die draait tijdens finalizeTree — de element-tree is dan gelockt. De video-editor luistert live mee via een ValueListenableBuilder<VideoPlayhead?> (de "Knip hier"-knop in lib/widgets/editors/video_slide_editor.dart); diens setState valt onder de lock en gooit de assertie. Beide dispose-paden raken het, dus de fix hoort in de bus.

Aanpak

Centraal in de bus: draait er een frame (schedulerPhase == persistentCallbacks), stel de clear dan uit tot ná het frame (addPostFrameCallback); anders synchroon wissen. De slideId-guard maakt de uitgestelde clear een no-op als een andere dia intussen zelf een afspeelpunt publiceert, zodat er geen vers afspeelpunt wordt weggegooid.

Regressietest: test/video_playhead_bus_test.dart (reproduceert de exacte assertie zonder de fix, groen met de fix).

## Symptoom Bij het weg-navigeren van een dia met (afspelende) video gooit Flutter tijdens het tekenen van het frame: ``` setState() or markNeedsBuild() called when widget tree was locked. This ValueListenableBuilder<VideoPlayhead?> widget cannot be marked as needing to build because the framework is locked. ``` De crash komt uit `VideoPlayheadBus.clearFor` → `_VideoPreviewState.dispose` tijdens `BuildOwner.finalizeTree`. ## Oorzaak `VideoPlayheadBus.clearFor` (`lib/widgets/slides/video_playhead_bus.dart`) zet `current.value = null`. Dat gebeurt vanuit `dispose()` van `_VideoPreviewState` én `_VideoEmbedPreviewState` (`lib/widgets/slides/previews/media_previews_video.dart`), die draait tijdens `finalizeTree` — de element-tree is dan gelockt. De video-editor luistert live mee via een `ValueListenableBuilder<VideoPlayhead?>` (de "Knip hier"-knop in `lib/widgets/editors/video_slide_editor.dart`); diens `setState` valt onder de lock en gooit de assertie. Beide dispose-paden raken het, dus de fix hoort in de bus. ## Aanpak Centraal in de bus: draait er een frame (`schedulerPhase == persistentCallbacks`), stel de clear dan uit tot ná het frame (`addPostFrameCallback`); anders synchroon wissen. De `slideId`-guard maakt de uitgestelde clear een no-op als een andere dia intussen zelf een afspeelpunt publiceert, zodat er geen vers afspeelpunt wordt weggegooid. Regressietest: `test/video_playhead_bus_test.dart` (reproduceert de exacte assertie zonder de fix, groen met de fix).

brenno referenced this issue from a commit

2026-08-02 09:58:58 +00:00

fix(video): stel afspeelpunt-clear uit buiten de gelockte widgettree (#1114)

brenno referenced this issue from a pull request that will close it,

2026-08-02 09:59:18 +00:00

fix(video): stel afspeelpunt-clear uit buiten de gelockte widgettree (#1114) #1120

brenno closed this issue

2026-08-02 09:59:30 +00:00

brenno referenced this issue from a commit

2026-08-02 09:59:31 +00:00

Merge pull request 'fix(video): stel afspeelpunt-clear uit buiten de gelockte widgettree (#1114)' (#1120) from fix/video-playhead-clear-locked-tree-1114 into main