From 09db56ba9343c597340b2c7715f43ff9b10826f6 Mon Sep 17 00:00:00 2001
From: Domenic Denicola <d@domenic.me>
Date: Mon, 30 Sep 2024 13:18:31 +0900
Subject: [PATCH] Reorder promise resolutions for navigation API

The API method tracker finished promise vs. the navigation transition finished promise ordering did not match the tests and Chromium implementation. There's no real argument for one order vs. the other, so update the spec to match.

Closes #10646.
---
 source | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/source b/source
index fd018b599ac..6d7b20c4cd5 100644
--- a/source
+++ b/source
@@ -95198,6 +95198,9 @@ interface <dfn interface>NavigationDestination</dfn> {
        <li><p><span data-x="concept-event-fire">Fire an event</span> named <code
        data-x="event-navigatesuccess">navigatesuccess</code> at <var>navigation</var>.</p></li>
 
+       <li><p>If <var>apiMethodTracker</var> is non-null, then <span>resolve the finished
+       promise</span> for <var>apiMethodTracker</var>.</p></li>
+
        <li><p>If <var>navigation</var>'s <span
        data-x="concept-Navigation-transition">transition</span> is not null, then resolve
        <var>navigation</var>'s <span data-x="concept-Navigation-transition">transition</span>'s
@@ -95206,9 +95209,6 @@ interface <dfn interface>NavigationDestination</dfn> {
 
        <li><p>Set <var>navigation</var>'s <span
        data-x="concept-Navigation-transition">transition</span> to null.</p></li>
-
-       <li><p>If <var>apiMethodTracker</var> is non-null, then <span>resolve the finished
-       promise</span> for <var>apiMethodTracker</var>.</p></li>
       </ol>
 
       <p>and the following failure steps given reason <var>rejectionReason</var>:</p>
@@ -95238,6 +95238,9 @@ interface <dfn interface>NavigationDestination</dfn> {
        <code>ErrorEvent</code>, with additional attributes initialized according to
        <var>errorInfo</var>.</p></li>
 
+       <li><p>If <var>apiMethodTracker</var> is non-null, then <span>reject the finished
+       promise</span> for <var>apiMethodTracker</var> with <var>rejectionReason</var>.</p></li>
+
        <li><p>If <var>navigation</var>'s <span
        data-x="concept-Navigation-transition">transition</span> is not null, then reject
        <var>navigation</var>'s <span data-x="concept-Navigation-transition">transition</span>'s
@@ -95246,9 +95249,6 @@ interface <dfn interface>NavigationDestination</dfn> {
 
        <li><p>Set <var>navigation</var>'s <span
        data-x="concept-Navigation-transition">transition</span> to null.</p></li>
-
-       <li><p>If <var>apiMethodTracker</var> is non-null, then <span>reject the finished
-       promise</span> for <var>apiMethodTracker</var> with <var>rejectionReason</var>.</p></li>
       </ol>
      </li>
     </ol>