Open Bug 646370 Opened 9 years ago Updated 2 years ago

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

Categories

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

defect

Tracking

()

People

(Reporter: gaubugzilla, Unassigned)

References

Details

(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: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
You need to log in before you can comment on or make changes to this bug.