From 8ce9baf92003bdc0a1bedc2be20c58c611e12d74 Mon Sep 17 00:00:00 2001
From: Prateek Sunal <prtksunal@gmail.com>
Date: Wed, 26 Feb 2025 13:59:03 +0530
Subject: [PATCH] fix: stop then dispose for now

---
 mobile/lib/ui/viewer/file/video_widget_native.dart | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/mobile/lib/ui/viewer/file/video_widget_native.dart b/mobile/lib/ui/viewer/file/video_widget_native.dart
index 5da199b965..a74cba0132 100644
--- a/mobile/lib/ui/viewer/file/video_widget_native.dart
+++ b/mobile/lib/ui/viewer/file/video_widget_native.dart
@@ -160,6 +160,11 @@ class _VideoWidgetNativeState extends State<VideoWidgetNative>
 
   @override
   void dispose() {
+    _subscription?.cancel();
+    _controller?.stop();
+    _controller?.dispose();
+    _controller = null;
+
     //https://github.com/fluttercandies/flutter_photo_manager/blob/8afba2745ebaac6af8af75de9cbded9157bc2690/README.md#clear-caches
     if (_shouldClearCache) {
       _logger.info("Clearing cache");
@@ -181,15 +186,12 @@ class _VideoWidgetNativeState extends State<VideoWidgetNative>
     removeCallBack(widget.file);
     _progressNotifier.dispose();
     WidgetsBinding.instance.removeObserver(this);
-    _subscription?.cancel();
     _isPlaybackReady.dispose();
     _showControls.dispose();
     _isSeeking.removeListener(_seekListener);
     _isSeeking.dispose();
     _debouncer.cancelDebounceTimer();
     _elTooltipController.dispose();
-    _controller?.dispose();
-    _controller = null;
     super.dispose();
   }