onProgress + e10s has wrong assertions

RESOLVED FIXED in Firefox 51

Status

()

Core
Networking: HTTP
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: baku, Assigned: baku)

Tracking

50 Branch
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

(Whiteboard: [necko-active])

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Created attachment 8785779 [details] [diff] [review]
progress.patch

It's easy to make a debug build to crash sending more data than what Content-Length header says.
Attachment #8785779 - Flags: review?(jduell.mcbugs)
Whiteboard: [necko-active]
Comment on attachment 8785779 [details] [diff] [review]
progress.patch

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

Parent code (nsHttpChannel) does a NS_WARNING for this case already, so I think we're good to remove the asserts here.
Attachment #8785779 - Flags: review?(jduell.mcbugs) → review+

Comment 2

2 years ago
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f03043a9f48
Remove onprogress assertions in HttpChannelChild, r=jduell

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6f03043a9f48
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.