1802677 - [wpt-sync] Sync PR 37133 - Revert "Soft navigations timestamp test clarifications and cleanups"

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Yuki Shiino <[e-mail]> wrote:

Revert "Soft navigations timestamp test clarifications and cleanups"

This reverts commit 8f8f242e2ceb5fa1302b7225785cf48665fca152.

Reason for revert: Suspicious about causing flaky test failures of:
external/wpt/soft-navigation-heuristics/navigation-api-hash.tentative.html
external/wpt/soft-navigation-heuristics/navigation-api-rejected.tentative.html
external/wpt/soft-navigation-heuristics/navigation-api.tentative.html
since:
https://ci.chromium.org/p/chromium/builders/ci/mac12-arm64-rel-tests/4047

Original change's description:

Soft navigations timestamp test clarifications and cleanups

Change-Id: Ia1c78427f8ec523721b77043990fb5cf63f8984c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4050662
Commit-Queue: Yoav Weiss \<[e-mail]>
Reviewed-by: Ian Clelland \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1075279}

Change-Id: I125fd6665aecb592587abd9c34b6f0df8946480e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4054265
Auto-Submit: Yuki Shiino \<[e-mail]>
Bot-Commit: Rubber Stamper \<[e-mail]>
Owners-Override: Yuki Shiino \<[e-mail]>
Commit-Queue: Rubber Stamper \<[e-mail]>
Cr-Commit-Position: refs/heads/main@{#1075452}

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 37133 applied with additional changes from upstream: f5e9057823b422dd24689d7a36ac0d3a2266668d, 85bcd6595ff523338d246d1bab435d03105f8a49

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=71667cac3a53325271c14acea5aa7017470577bc

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

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

Total 23 tests and 1 subtests

Status Summary

Firefox

OK : 1
PASS : 1
FAIL : 1
TIMEOUT: 36
ERROR : 1[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-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt] 4[GitHub]

Chrome

OK : 17
PASS : 10
FAIL : 7
TIMEOUT: 12

Safari

TIMEOUT: 38
ERROR : 4

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /soft-navigation-heuristics/back.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/dropped-entries.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/hash.tentative.html [wpt.fyi]: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
  • Detect hashchange event.: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
• /soft-navigation-heuristics/image-lcp-followed-by-image-softnav-lcp.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/image-lcp-followed-by-text-softnav-lcp.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/navigate-child.html [wpt.fyi]
  • Test that a navigated child window doesn't crash: FAIL [Gecko-windows10-32-2004-qr-debug, Gecko-windows10-32-2004-qr-opt, Gecko-windows10-64-2004-qr-debug, Gecko-windows10-64-2004-qr-opt, GitHub], PASS [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] (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/navigation-api-hash.tentative.html [wpt.fyi]: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
  • Test soft navigation with the Navigation API: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
• /soft-navigation-heuristics/navigation-api-preventDefault.tentative.html [wpt.fyi]: ERROR (Chrome: OK, Safari: ERROR)
• /soft-navigation-heuristics/navigation-api-rejected.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Test intercepted and rejected navigate event: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/navigation-api.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Test soft navigation with the Navigation API: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/popstate.tentative.html [wpt.fyi]: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
  • A soft navigation that uses a same-document initiated popstate event is recognized by SoftNavigationHeuristics: TIMEOUT (Chrome: TIMEOUT, Safari: TIMEOUT)
• /soft-navigation-heuristics/replacestate-null-then-push.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect soft navigation with replaceState that has a null URL, then
    pushState with the URL.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/replacestate.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect soft navigation with replaceState.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/soft-navigation-detection-main-descendent.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect soft navigation adding content to a main descendent.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/soft-navigation-detection-non-main.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/soft-navigation-detection-web-component-lifecycle.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/soft-navigation-detection.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/soft-navigation-no-url.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Test that a soft navigation is not detected when a URL is not passed to the history API.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/text-lcp-followed-by-image-softnav-lcp.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
• /soft-navigation-heuristics/text-lcp-followed-by-text-softnav-lcp.tentative.html [wpt.fyi]: TIMEOUT (Chrome: OK, Safari: TIMEOUT)
  • Detect simple soft navigation.: TIMEOUT (Chrome: FAIL, Safari: TIMEOUT)
• /soft-navigation-heuristics/navigation-api-back.tentative.html [wpt.fyi]: ERROR (Chrome: TIMEOUT, Safari: ERROR)
• /soft-navigation-heuristics/navigation-api-forward.tentative.html [wpt.fyi]: ERROR (Chrome: TIMEOUT, Safari: ERROR)
• /soft-navigation-heuristics/navigation-api-traverseto.tentative.html [wpt.fyi]: ERROR (Chrome: TIMEOUT, Safari: ERROR)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd9deeac6d07
[wpt PR 37133] - Revert "Soft navigations timestamp test clarifications and cleanups", a=testonly

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/fd9deeac6d07