Closed Bug 1405976 Opened 4 years ago Closed 4 years ago

PartiallySeekableInputStream should take the ownership of the underlying stream

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: baku, Assigned: baku)

Details

Attachments

(1 file)

This is needed to avoid the use of the underlying stream after the wrapping.
Unfortunately this is not particularly useful in HttpBaseChannel::ExplicitSetUploadStream because it receives just a regular pointer and not a smart one.
Attached patch cc.patchSplinter Review
Attachment #8915497 - Flags: review?(kyle)
Attachment #8915497 - Flags: review?(kyle) → review+
Priority: -- → P3
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/09796501dad6
PartiallySeekableInputStream must take the ownership of the underlying stream, r=qdot
https://hg.mozilla.org/mozilla-central/rev/09796501dad6
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.