Bug 1790375 - [bidi] Listen to _fetchError when waiting for navigation r=webdriver-reviewers,whimboo

Depends on D197586

Differential Revision: https://phabricator.services.mozilla.com/D197587

remote/webdriver-bidi/modules/root/browsingContext.sys.mjs

@@ -1422,12 +1422,13 @@ class BrowsingContextModule extends Module {
           shouldWaitForNavigationRequest = true;
         }
       };
-      const onResponseCompleted = (name, data) => {
+      const onNetworkRequestCompleted = (name, data) => {
         if (data.navigation) {
           resolveOnNetworkEvent();
         }
       };
 
+      // The network request can either end with _responseCompleted or _fetchError
       await this.messageHandler.eventsDispatcher.on(
         "network._beforeRequestSent",
         contextDescriptor,
@@ -1436,7 +1437,12 @@ class BrowsingContextModule extends Module {
       await this.messageHandler.eventsDispatcher.on(
         "network._responseCompleted",
         contextDescriptor,
-        onResponseCompleted
+        onNetworkRequestCompleted
+      );
+      await this.messageHandler.eventsDispatcher.on(
+        "network._fetchError",
+        contextDescriptor,
+        onNetworkRequestCompleted
       );
 
       unsubscribeNavigationListeners = async () => {
@@ -1445,11 +1451,15 @@ class BrowsingContextModule extends Module {
           contextDescriptor,
           onBeforeRequestSent
         );
-
         await this.messageHandler.eventsDispatcher.off(
           "network._responseCompleted",
           contextDescriptor,
-          onResponseCompleted
+          onNetworkRequestCompleted
+        );
+        await this.messageHandler.eventsDispatcher.off(
+          "network._fetchError",
+          contextDescriptor,
+          onNetworkRequestCompleted
         );
       };
     }