Closed Bug 1865142 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 43211 - Rewrite the way some history tests await the pass condition.

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
122 Branch
Tracking Flags:
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 43211 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/43211
Details from upstream follow.

L. David Baron <dbaron@chromium.org> wrote:

Rewrite the way some history tests await the pass condition.

Instead of expecting all history navigations to complete within 500ms
(which can fail in some scenarios such as debugging tools or slow test
setups), have them poll for the success condition every 50ms (starting
immediately rather than from a function called from the opened window)
and then wait 500ms to confirm that the success condition hasn't
changed.

Fixed: 1502718
Bug: 1486131
Change-Id: I818d06efa53f914619dc935a310362cb0c983fcc

Reviewed-on: https://chromium-review.googlesource.com/5038190
WPT-Export-Revision: 548e9a2b5d438b8476a6fb84984cc476dae14bb5

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 16 tests and 1 subtests

Status Summary

Firefox

OK : 11[Gecko-linux1804-64-qr-debug] 12[Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug] 14[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt, GitHub]
PASS : 10[Gecko-linux1804-64-qr-debug] 11[Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug] 13[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt, GitHub]
FAIL : 1
TIMEOUT: 2[Gecko-linux1804-64-qr-debug] 4[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub]

Chrome

OK : 14
PASS : 13
FAIL : 1
TIMEOUT: 4

Safari

TIMEOUT: 32

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

  • /html/browsers/history/the-history-interface/history_go_zero.html [wpt.fyi]: OK [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], SKIP [Gecko-linux1804-64-qr-debug] (Chrome: OK, Safari: TIMEOUT)
  • /html/browsers/history/the-history-interface/traverse_the_history_write_after_load_1.html [wpt.fyi]: SKIP [Gecko-linux1804-64-qr-debug], TIMEOUT [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub] (Chrome: TIMEOUT, Safari: TIMEOUT)
  • /html/browsers/history/the-history-interface/traverse_the_history_write_onload_1.html [wpt.fyi]: OK [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt, GitHub], SKIP [Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug] (Chrome: OK, Safari: TIMEOUT)
  • /html/browsers/history/the-history-interface/traverse_the_history_write_onload_2.html [wpt.fyi]: OK [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt, GitHub], SKIP [Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug] (Chrome: OK, Safari: TIMEOUT)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ddc043d474f6 [wpt PR 43211] - Rewrite the way some history tests await the pass condition., a=testonly https://hg.mozilla.org/integration/autoland/rev/b7239c4ad91b [wpt PR 43211] - Update wpt metadata, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/29bce4041b49 [wpt PR 43211] - Rewrite the way some history tests await the pass condition., a=testonly https://hg.mozilla.org/integration/autoland/rev/6cf545731249 [wpt PR 43211] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/29bce4041b49
https://hg.mozilla.org/mozilla-central/rev/6cf545731249

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox122: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.