Closed Bug 1766538 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 33803 - Reland "Close a MessagePort if it is created in a destroyed context."

Categories

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

task

Tracking

()

RESOLVED FIXED
102 Branch
Tracking Status
firefox102 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Marijn Kruisselbrink <mek@chromium.org> wrote:

Reland "Close a MessagePort if it is created in a destroyed context."

This is a reland of commit 068f13cc5aa5f7a6e9faf28d8731275e64cb657b

Original change's description:

Close a MessagePort if it is created in a destroyed context.

MessagePort assumes it is only destroyed either after ContextDestroyed,
or after the port has been closed explicitly. As it turns out ports that
were created in an already detached iframe would violate this invariant,
causing issues.

Bug: 1228661
Change-Id: Ib1abce15f1d1d15f044de19fe0534767db488af0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3561845
Reviewed-by: Jeremy Roman \<jbroman@chromium.org>
Commit-Queue: Marijn Kruisselbrink \<mek@chromium.org>
Cr-Commit-Position: refs/heads/main@{#988859}

Bug: 1228661
Change-Id: Ifc5ec866678667b0d81438e2a2c8e5ada6e19d8c

Reviewed-on: https://chromium-review.googlesource.com/3609249
WPT-Export-Revision: 56ba7aacddbc9b55fdb96ab2a4e8bc0be9e06c89

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

CI Results

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

Total 1 tests and 2 subtests

Status Summary

Firefox

OK : 1
PASS: 2

Chrome

OK : 1
PASS: 2

Safari

OK : 1
PASS: 2

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b4ee042622cb
[wpt PR 33803] - Reland "Close a MessagePort if it is created in a destroyed context.", a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.