Workaround for seeking to somewhere in paused state results in infinite loading on android

Author: timoxd7

lib/src/cupertino/cupertino_controls.dart

@@ -9,6 +9,7 @@ import 'package:chewie/src/chewie_player.dart';
 import 'package:chewie/src/chewie_progress_colors.dart';
 import 'package:chewie/src/cupertino/cupertino_progress_bar.dart';
 import 'package:chewie/src/cupertino/widgets/cupertino_options_dialog.dart';
+import 'package:chewie/src/helpers/cancel_token.dart';
 import 'package:chewie/src/helpers/utils.dart';
 import 'package:chewie/src/models/option_item.dart';
 import 'package:chewie/src/models/subtitle_model.dart';
@@ -808,12 +809,19 @@ class _CupertinoControlsState extends State<CupertinoControls>
     }
   }
 
+  CancelToken? _updateStateCancelToken;
+
   void _updateState() {
     if (!mounted) return;
 
     late final bool buffering;
 
     if (Platform.isAndroid) {
+      if (_updateStateCancelToken != null) {
+        _updateStateCancelToken!.cancel();
+        _updateStateCancelToken = null;
+      }
+
       if (controller.value.isBuffering) {
         // -> Check if we actually buffer, as android has a bug preventing to
         //    get the correct buffering state from this single bool.
@@ -835,6 +843,19 @@ class _CupertinoControlsState extends State<CupertinoControls>
         // -> No buffering
         buffering = false;
       }
+
+      // We need to check again later, as the buffering state might change
+      // without notice. This is due to no listener is called as buffering is
+      // always true.
+      if (buffering) {
+        final CancelToken cancelToken = CancelToken();
+        _updateStateCancelToken = cancelToken;
+
+        Future.delayed(const Duration(milliseconds: 100), () {
+          if (cancelToken.isCancelled) return;
+          _updateState();
+        });
+      }
     } else {
       buffering = controller.value.isBuffering;
     }

lib/src/helpers/cancel_token.dart

@@ -0,0 +1,11 @@
+class CancelToken {
+  bool _isCancelled = false;
+
+  /// Cancel the request.
+  void cancel() {
+    _isCancelled = true;
+  }
+
+  /// Check if the request is cancelled.
+  bool get isCancelled => _isCancelled;
+}

lib/src/material/material_controls.dart

@@ -5,6 +5,7 @@ import 'package:chewie/src/center_play_button.dart';
 import 'package:chewie/src/center_seek_button.dart';
 import 'package:chewie/src/chewie_player.dart';
 import 'package:chewie/src/chewie_progress_colors.dart';
+import 'package:chewie/src/helpers/cancel_token.dart';
 import 'package:chewie/src/helpers/utils.dart';
 import 'package:chewie/src/material/color_compat_extensions.dart';
 import 'package:chewie/src/material/material_progress_bar.dart';
@@ -643,12 +644,19 @@ class _MaterialControlsState extends State<MaterialControls>
     }
   }
 
+  CancelToken? _updateStateCancelToken;
+
   void _updateState() {
     if (!mounted) return;
 
     late final bool buffering;
 
     if (Platform.isAndroid) {
+      if (_updateStateCancelToken != null) {
+        _updateStateCancelToken!.cancel();
+        _updateStateCancelToken = null;
+      }
+
       if (controller.value.isBuffering) {
         // -> Check if we actually buffer, as android has a bug preventing to
         //    get the correct buffering state from this single bool.
@@ -670,6 +678,19 @@ class _MaterialControlsState extends State<MaterialControls>
         // -> No buffering
         buffering = false;
       }
+
+      // We need to check again later, as the buffering state might change
+      // without notice. This is due to no listener is called as buffering is
+      // always true.
+      if (buffering) {
+        final CancelToken cancelToken = CancelToken();
+        _updateStateCancelToken = cancelToken;
+
+        Future.delayed(const Duration(milliseconds: 100), () {
+          if (cancelToken.isCancelled) return;
+          _updateState();
+        });
+      }
     } else {
       buffering = controller.value.isBuffering;
     }

lib/src/material/material_desktop_controls.dart

@@ -5,6 +5,7 @@ import 'package:chewie/src/animated_play_pause.dart';
 import 'package:chewie/src/center_play_button.dart';
 import 'package:chewie/src/chewie_player.dart';
 import 'package:chewie/src/chewie_progress_colors.dart';
+import 'package:chewie/src/helpers/cancel_token.dart';
 import 'package:chewie/src/helpers/utils.dart';
 import 'package:chewie/src/material/color_compat_extensions.dart';
 import 'package:chewie/src/material/material_progress_bar.dart';
@@ -579,12 +580,19 @@ class _MaterialDesktopControlsState extends State<MaterialDesktopControls>
     }
   }
 
+  CancelToken? _updateStateCancelToken;
+
   void _updateState() {
     if (!mounted) return;
 
     late final bool buffering;
 
     if (Platform.isAndroid) {
+      if (_updateStateCancelToken != null) {
+        _updateStateCancelToken!.cancel();
+        _updateStateCancelToken = null;
+      }
+
       if (controller.value.isBuffering) {
         // -> Check if we actually buffer, as android has a bug preventing to
         //    get the correct buffering state from this single bool.
@@ -606,6 +614,19 @@ class _MaterialDesktopControlsState extends State<MaterialDesktopControls>
         // -> No buffering
         buffering = false;
       }
+
+      // We need to check again later, as the buffering state might change
+      // without notice. This is due to no listener is called as buffering is
+      // always true.
+      if (buffering) {
+        final CancelToken cancelToken = CancelToken();
+        _updateStateCancelToken = cancelToken;
+
+        Future.delayed(const Duration(milliseconds: 100), () {
+          if (cancelToken.isCancelled) return;
+          _updateState();
+        });
+      }
     } else {
       buffering = controller.value.isBuffering;
     }