Skip to content

Conversation

@farre
Copy link

@farre farre commented Nov 13, 2025

All these tests, more or less, fail because of two reason that work together:

  • They expect Promise.all behavior for calling #commit-a-navigate-event[1] when it's specced that #wait-for-all[2] should be used.
  • #commit-a-navigate-event[3] falls through after asynchronously calling #resume-applying-the-traverse-history-step, where the same #wait-for-all can finish before the resumption of #apply-the-history-step[5] finishes.

See the discussion in:

[1] https://html.spec.whatwg.org/#commit-a-navigate-event
[2] https://webidl.spec.whatwg.org/#wait-for-all
[3] https://html.spec.whatwg.org/#navigate-event-firing:resume-applying-the-traverse-history-step
[4] https://html.spec.whatwg.org/#resume-applying-the-traverse-history-step
[5] https://html.spec.whatwg.org/#apply-the-history-step

All these tests, more or less, fail because of two reason that work
together:
* They expect Promise.all behavior for calling #commit-a-navigate-event[1]
  when it's specced that #wait-for-all[2] should be used.
* #commit-a-navigate-event[3] falls through after asynchronously calling
  #resume-applying-the-traverse-history-step, where the same #wait-for-all
  can finish before the resumption of #apply-the-history-step[5] finishes.

See the discussion in:
* whatwg/html#11814
* whatwg/html#11845

[1] https://html.spec.whatwg.org/#commit-a-navigate-event
[2] https://webidl.spec.whatwg.org/#wait-for-all
[3] https://html.spec.whatwg.org/#navigate-event-firing:resume-applying-the-traverse-history-step
[4] https://html.spec.whatwg.org/#resume-applying-the-traverse-history-step
[5] https://html.spec.whatwg.org/#apply-the-history-step
@farre farre added the do not merge yet Disable auto-merge label Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet Disable auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant