From 5a6b043fb199593bb6b75d6969a756fefac1a325 Mon Sep 17 00:00:00 2001
From: Jackiu <jackiu1997@outlook.com>
Date: Sat, 6 May 2023 15:32:52 +0800
Subject: [PATCH] Fix: fix windows player black screen bug

---
 .../video_player/video_controller.dart        | 20 ++++---------------
 .../widgets/video_player/video_player.dart    |  2 --
 .../video_player/video_player_provider.dart   | 18 -----------------
 3 files changed, 4 insertions(+), 36 deletions(-)
 delete mode 100644 lib/modules/live_play/widgets/video_player/video_player_provider.dart

diff --git a/lib/modules/live_play/widgets/video_player/video_controller.dart b/lib/modules/live_play/widgets/video_player/video_controller.dart
index 44e2edf1..5d1d6b3a 100644
--- a/lib/modules/live_play/widgets/video_player/video_controller.dart
+++ b/lib/modules/live_play/widgets/video_player/video_controller.dart
@@ -151,8 +151,6 @@ class VideoController with ChangeNotifier {
         mobileController?.videoPlayerController?.value.isPip ?? false;
   }
 
-  VideoPlayerProvider? controllerProvider;
-
   // Danmaku player control
   final danmakuController = BarrageWallController();
   final hideDanmaku = false.obs;
@@ -302,10 +300,7 @@ class VideoController with ChangeNotifier {
         Navigator.push(
           context,
           MaterialPageRoute(
-            builder: (context) => DesktopFullscreen(
-              controller: controllerProvider!.controller,
-              child: controllerProvider!.child,
-            ),
+            builder: (context) => DesktopFullscreen(controller: this),
           ),
         );
       } else {
@@ -353,10 +348,7 @@ class VideoController with ChangeNotifier {
         Navigator.push(
           context,
           MaterialPageRoute(
-            builder: (context) => DesktopFullscreen(
-              controller: controllerProvider!.controller,
-              child: controllerProvider!.child,
-            ),
+            builder: (context) => DesktopFullscreen(controller: this),
           ),
         );
       } else {
@@ -486,14 +478,10 @@ class _MobileFullscreenState extends State<MobileFullscreen>
 }
 
 class DesktopFullscreen extends StatelessWidget {
-  const DesktopFullscreen({
-    Key? key,
-    required this.controller,
-    required this.child,
-  }) : super(key: key);
+  const DesktopFullscreen({Key? key, required this.controller})
+      : super(key: key);
 
   final VideoController controller;
-  final Widget child;
 
   @override
   Widget build(BuildContext context) {
diff --git a/lib/modules/live_play/widgets/video_player/video_player.dart b/lib/modules/live_play/widgets/video_player/video_player.dart
index c4e8da91..c358b9b8 100644
--- a/lib/modules/live_play/widgets/video_player/video_player.dart
+++ b/lib/modules/live_play/widgets/video_player/video_player.dart
@@ -7,8 +7,6 @@ import 'package:pure_live/common/index.dart';
 import 'package:pure_live/modules/live_play/widgets/video_player/video_controller.dart';
 import 'package:pure_live/modules/live_play/widgets/video_player/video_controller_panel.dart';
 
-import 'video_player_provider.dart';
-
 class VideoPlayer extends StatefulWidget {
   final VideoController controller;
 
diff --git a/lib/modules/live_play/widgets/video_player/video_player_provider.dart b/lib/modules/live_play/widgets/video_player/video_player_provider.dart
deleted file mode 100644
index 2a85ed6e..00000000
--- a/lib/modules/live_play/widgets/video_player/video_player_provider.dart
+++ /dev/null
@@ -1,18 +0,0 @@
-import 'package:flutter/material.dart';
-
-import 'video_controller.dart';
-
-///Widget which is used to inherit BetterPlayerController through widget tree.
-class VideoPlayerProvider extends InheritedWidget {
-  const VideoPlayerProvider({
-    Key? key,
-    required this.controller,
-    required Widget child,
-  }) : super(key: key, child: child);
-
-  final VideoController controller;
-
-  @override
-  bool updateShouldNotify(VideoPlayerProvider oldWidget) =>
-      controller != oldWidget.controller;
-}