Closed Bug 1814644 Opened 2 years ago Closed 2 years ago

Remove GetIncumbentGlobal use in dom/streams

Categories

(Core :: DOM: Streams, task)

task

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz)

Details

Attachments

(1 file)

One may want to create a stream in an async C++ callback for example, in which case there won't be any relevant JS context in the call stack.

That was the only case that Promise::CreateInfallible may possibly get nullptr in dom/streams case, so it now just asserts the existence of nsIGlobalObject.

Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cb1d91993dd1 Remove GetIncumbentGlobal use in SetUpReadableByteStreamController r=smaug
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: