Closed Bug 645281 Opened 10 years ago Closed 10 years ago

nsACProxyListener::OnDataAvailable is broken

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: sicking)

Details

Attachments

(1 file)

It needs to either throw or consume the data from the stream: that's the nsIStreamListener contract.  It's current behavior causes asserts if the server responds to a preflight by sending a message with a body...

Jonas, you're reworking this code anyway, right?
OS: Mac OS X → All
Hardware: x86 → All
Attached patch Patch to fixSplinter Review
Assignee: nobody → jonas
Attachment #523156 - Flags: review?(bzbarsky)
Comment on attachment 523156 [details] [diff] [review]
Patch to fix

s/DummyStreamReaderFunc/NS_DiscardSegment/ (after including nsStreamUtils.h), and r=me
Attachment #523156 - Flags: review?(bzbarsky) → review+
Checked in. Thanks for finding this and for reviewing!

http://hg.mozilla.org/mozilla-central/rev/bc035b7a2aab
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.