fix: avoid premature context cancellation when navigation returns a value

Replace the SetContext-based NotifyDestroyed call (which raced with async
response processing) with a Frame.FrameNavigated subscription. Chrome
sends callFunctionOn responses before Page.frameNavigated, so signalling
destruction from FrameNavigated guarantees valid return values are
already resolved before cancellation fires.

Fixes ShouldNotThrowAnErrorWhenEvaluationDoesANavigation hang introduced
by the previous context-destruction fix.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: kblok

lib/PuppeteerSharp/IsolatedWorld.cs

@@ -47,6 +47,11 @@ public IsolatedWorld(
 
             _detached = false;
             FrameUpdated();
+
+            if (Frame != null)
+            {
+                Frame.FrameNavigated += Frame_FrameNavigated;
+            }
         }
 
         /// <inheritdoc/>
@@ -211,6 +216,12 @@ internal void Detach()
         {
             _detached = true;
             Client.MessageReceived -= Client_MessageReceived;
+
+            if (Frame != null)
+            {
+                Frame.FrameNavigated -= Frame_FrameNavigated;
+            }
+
             TaskManager.TerminateAll(new PuppeteerException("waitForFunction failed: frame got detached."));
         }
 
@@ -286,18 +297,20 @@ internal void SetContext(ExecutionContext context)
                 throw new ArgumentNullException(nameof(context));
             }
 
-            // Cancel any pending evaluations on the old context before replacing it.
-            // This handles the edge case where a new context arrives without an explicit
-            // executionContextsCleared/Destroyed event (e.g., Chrome reusing a context ID).
-            var oldContext = _context;
             _context = context;
-            oldContext?.NotifyDestroyed();
-
             _contextBindings.Clear();
             _contextResolveTaskWrapper.TrySetResult(context);
             TaskManager.RerunAll();
         }
 
+        private void Frame_FrameNavigated(object sender, FrameNavigatedEventArgs e)
+        {
+            if (!e.NavigatedWithinDocument)
+            {
+                _context?.NotifyDestroyed();
+            }
+        }
+
         private async void Client_MessageReceived(object sender, MessageEventArgs e)
         {
             try