Fix some UnobservedTaskException errors

Removed method which was not called anymore. Added override method which handles the StateMaanger.Exited use case explicitly.
drawcode · Dec 20, 2021 · aa5fb43 · aa5fb43
1 parent e83a4ac
commit aa5fb43
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/lib/PuppeteerSharp/States/DisposedState.cs b/lib/PuppeteerSharp/States/DisposedState.cs
@@ -9,21 +9,19 @@ public DisposedState(StateManager stateManager) : base(stateManager)
         {
         }
 
-        public void EnterFrom(LauncherBase p, State fromState)
+        public override Task EnterFromAsync(LauncherBase p, State fromState, TimeSpan timeSpan)
         {
-            if (!StateManager.TryEnter(p, fromState, this))
+            if (fromState == StateManager.Exited)
             {
-                // Delegate Dispose to current state, because it has already changed since
-                // transition to this state was initiated.
-                StateManager.CurrentState.Dispose(p);
+                return null;
             }
-            else if (fromState != StateManager.Exited)
-            {
-                Kill(p);
 
-                p.ExitCompletionSource.TrySetException(new ObjectDisposedException(p.ToString()));
-                p.TempUserDataDir?.Dispose();
-            }
+            Kill(p);
+
+            p.ExitCompletionSource.TrySetException(new ObjectDisposedException(p.ToString()));
+            p.TempUserDataDir?.Dispose();
+
+            return null;
         }
 
         public override Task StartAsync(LauncherBase p) => throw new ObjectDisposedException(p.ToString());