feat: do not fail navigation command if was canceled by another navigation

Author: sadym-chromium

src/bidiMapper/modules/context/BrowsingContextImpl.ts

@@ -1033,9 +1033,11 @@ export class BrowsingContextImpl {
 
     if (result instanceof NavigationResult) {
       if (
-        // TODO: check after decision on the spec is done:
-        //  https://github.com/w3c/webdriver-bidi/issues/799.
-        result.eventName === NavigationEventName.NavigationAborted ||
+        // If the navigation was committed and then another navigation started, the
+        // command should be considered as successfully finished.
+        // Details:
+        // * https://github.com/w3c/webdriver-bidi/issues/763
+        // * https://github.com/w3c/webdriver-bidi/issues/799
         result.eventName === NavigationEventName.NavigationFailed
       ) {
         throw new UnknownErrorException(result.message ?? 'unknown exception');

tests/browsing_context/test_navigate.py

@@ -45,9 +45,11 @@ async def test_browsingContext_navigateWaitInteractive_redirect(
     assert messages == [
         AnyExtending({
             'id': command_id,
-            'error': 'unknown error',
-            'message': 'navigation canceled by concurrent navigation',
-            'type': 'error',
+            'result': {
+                'navigation': 'stable_0',
+                'url': initial_url
+            },
+            'type': 'success',
         }),
         {
             'method': 'browsingContext.navigationAborted',