Closed Bug 1373555 Opened 3 years ago Closed 2 years ago

Move the Fetch consume body login in a separate class

Categories

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

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: baku, Assigned: baku)

References

Details

Attachments

(3 files, 1 obsolete file)

No description provided.
Attached patch part 1 - separate files (obsolete) — Splinter Review
Attachment #8878373 - Flags: review?(bkelly)
Comment on attachment 8878373 [details] [diff] [review]
part 1 - separate files

Review of attachment 8878373 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry, can you rebase this patch without the hg copy?  Its making it hard to read the diff in splinter.
Attachment #8878373 - Flags: review?(bkelly)
Attachment #8878373 - Attachment is obsolete: true
Attachment #8878925 - Flags: review?(bkelly)
Comment on attachment 8878925 [details] [diff] [review]
part 1 - separate files

Review of attachment 8878925 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Attachment #8878925 - Flags: review?(bkelly) → review+
Attachment #8878374 - Flags: review?(bkelly) → review+
Attachment #8878375 - Flags: review?(bkelly) → review+
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/95263ffce514
Move the Fetch consume body login in a separate class - part 1 - separate files, r=bkelly
https://hg.mozilla.org/integration/mozilla-inbound/rev/0092bff5f1c5
Move the Fetch consume body login in a separate class - part 2 - Rename FetchBodyWrapper to FetchBodyConsumer, r=bkelly
https://hg.mozilla.org/integration/mozilla-inbound/rev/f5b120a0b480
Move the Fetch consume body login in a separate class - part 3 - Move the consuming body logic from FetchBody to FetchBodyConsumer, r=bkelly
Depends on: 1376282
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.