Closed Bug 1759572 Opened 5 months ago Closed 3 months ago

nsMultiplexInputStream::AsyncWaitLengthHelper should be threadsafe

Categories

(Core :: XPCOM, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
102 Branch
Tracking Status
firefox102 --- fixed

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(1 file)

No description provided.

This works only because the only implementation of nsIAsyncInputStreamLength
happened to only run on the current thread. Bug 1754004, changes this, and requires the reference to be threadsafe.

Depends on D141036

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:nika, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(nika)
Flags: needinfo?(bugmail)

Waiting on other bugs in the stack before landing

Flags: needinfo?(nika)
Flags: needinfo?(bugmail)
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c8ec86e0a33b
AsyncLengthWaitHelper should be threadsafe, r=asuth
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a15de342dab9
AsyncLengthWaitHelper should be threadsafe, r=asuth
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch

Backed out as they were linked to the backed out Bug 1754004

Backout link

Status: RESOLVED → REOPENED
Flags: needinfo?(nika)
Resolution: FIXED → ---
Target Milestone: 102 Branch → ---
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4abc98ed6b99
AsyncLengthWaitHelper should be threadsafe, r=asuth
Flags: needinfo?(nika)
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9b24b561698c
AsyncLengthWaitHelper should be threadsafe, r=asuth
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/83e2d1f0a115
AsyncLengthWaitHelper should be threadsafe, r=asuth
Flags: needinfo?(nika)
Status: REOPENED → RESOLVED
Closed: 3 months ago3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.