Open Bug 646370 Opened 13 years ago Updated 3 years ago

Stream listener should not be overwritten after triggering http-on-modify-request observers


(Core :: Networking: HTTP, defect, P5)





(Reporter: jwkbugzilla, Unassigned)



(Whiteboard: [necko-backlog])

Currently the code in nsHttpChannel::AsyncOpen() will first trigger http-on-modify-request observes and only then initialize mListener member. So if one of these observers decides to use nsITraceableChannel and set its own listener (which is the most obvious way to do it if you want to trace all requests) it won't work - the listener set by http-on-modify-request will simply be overwritten. Initialization of mListener member should be moved up before gHttpHandler->OnModifyRequest() call.
Whiteboard: [necko-backlog]
Bulk change to priority:
Priority: -- → P1
Bulk change to priority:
Priority: P1 → P3

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.