Note: There are a few cases of duplicates in user autocompletion which are being worked on.

nullhttptransaciton based spdystream does not finish

RESOLVED FIXED in mozilla15

Status

()

Core
Networking: HTTP
--
minor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mcmanus, Assigned: mcmanus)

Tracking

15 Branch
mozilla15
x86_64
Linux
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sdpy][http-conn])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
spdystream expects its transaction to produce an http request that it gateways into spdy. a nullHttpTransaction does not do that, and the spdystream will pretty much wait for it to do so until it is timed out (even though NullHttpTransaction has retured BASE_STREAM_CLOSED).

Null transactions only exist on nightly. I haven't been able to produce anything bad with this code path, but it explains why some objects in my log take longer than expected to go away. They might even effectively leak.
(Assignee)

Comment 1

5 years ago
Created attachment 624055 [details] [diff] [review]
patch 0
Assignee: nobody → mcmanus
Status: NEW → ASSIGNED
Attachment #624055 - Flags: review?(honzab.moz)
Comment on attachment 624055 [details] [diff] [review]
patch 0

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

r=honzan
Attachment #624055 - Flags: review?(honzab.moz) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/938102758cea
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/938102758cea
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [sdpy] → [sdpy][http-conn]
You need to log in before you can comment on or make changes to this bug.