Closed Bug 1759569 Opened 5 months ago Closed 3 months ago

Retain nsIInputStreamLength through NS_CloneInputStream

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)

This will help in bug 1754004 as in some cases we will want to clone a DataPipe, which is non-cloneable, and without this change length information would be lost.

In the future it might be worth making a more generic mechanism for efficiently cloning wrapper input streams with a fallback rather than hard-coding this interface.

This will help in bug 1754004 as in some cases we will want to clone a
DataPipe, which is non-cloneable, and without this change length information
would be lost.

In the future it might be worth making a more generic mechanism for efficiently
cloning wrapper input streams with a fallback rather than hard-coding this
interface.

Depends on D141034

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/bd3a2fcf4f0c
Retain nsIInputStreamLength through NS_CloneInputStream, r=asuth
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d70810d7b63a
Retain nsIInputStreamLength through NS_CloneInputStream, 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/fadb6cc0d81c
Retain nsIInputStreamLength through NS_CloneInputStream, r=asuth
Flags: needinfo?(nika)
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8d82066fa181
Retain nsIInputStreamLength through NS_CloneInputStream, r=asuth
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fc479c3e1811
Retain nsIInputStreamLength through NS_CloneInputStream, 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.