Closed Bug 1304017 Opened 3 years ago Closed 3 years ago

improve MultipartBlobImpl refcounting

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

MultipartBlobImpl always requires copying the incoming BlobImpl array.
This copying, and its associated refcounting, is unnecessary in many
cases, as we're just going to delete the copied-from array anyway.  We
might as well pass in ownership when creating the multipart blob and
avoid all the overhead associated with copying and refcounting.
Attachment #8792884 - Flags: review?(amarchesini)
Attachment #8792884 - Flags: review?(amarchesini) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/665d28dc815d
improve MultipartBlobImpl refcounting; r=baku
https://hg.mozilla.org/mozilla-central/rev/665d28dc815d
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.