Closed Bug 1945304 Opened 23 days ago Closed 21 days ago

[wpt-sync] Sync PR 50425 - Reland "Ensure AbortSignal.throwIfAborted doesn't modify the message of an abort error"

Categories

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

task

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Nate Chapin <japhet@chromium.org> wrote:

Reland "Ensure AbortSignal.throwIfAborted doesn't modify the message of an abort error"

This reverts commit 74320221ad24d0050a8baa031ca1d0e998ea7fc6.

Reason for revert: Not the cause of issues.chromium.org/issues/393009392

Original change's description:

Revert "Ensure AbortSignal.throwIfAborted doesn't modify the message of an abort error"

This reverts commit 0e0ef178e05e3f81b0de04c8f0197fe3ab608f41.

Reason for revert: Possible culprit for windows builder closure, see https://g-issues.chromium.org/issues/393009392.

Original change's description:

Ensure AbortSignal.throwIfAborted doesn't modify the message of an abort error

Fixed: 392413688
Change-Id: I3639c573cd7349eac1f9b09c427a5a52bccdda72
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6209470
Reviewed-by: Andrey Kosyakov \<caseq@chromium.org>
Commit-Queue: Nate Chapin \<japhet@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1412605}

Change-Id: Ifb785338a85e77e845a677940fc1353b74757b0f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6211537
Commit-Queue: Patti Lor \<patricialor@chromium.org>
Owners-Override: Patti Lor \<patricialor@chromium.org>
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Patti Lor \<patricialor@chromium.org>
Auto-Submit: Patti Lor \<patricialor@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1412694}

Change-Id: Ic42d7631d3ebe6a1c902e0763fdc3335f5262b33
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6221802
Commit-Queue: Andrey Kosyakov \<caseq@chromium.org>
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: Andrey Kosyakov \<caseq@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1414549}

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 8 tests

Status Summary

Firefox

OK : 2
PASS : 32
ERROR: 6

Chrome

OK : 2
PASS : 30
FAIL : 2
ERROR: 6

Safari

OK : 2
PASS : 32
ERROR: 6

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2bfd5a71c2bf [wpt PR 50425] - Reland "Ensure AbortSignal.throwIfAborted doesn't modify the message of an abort error", a=testonly
Status: NEW → RESOLVED
Closed: 21 days ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.