Open
Bug 646370
Opened 14 years ago
Updated 4 years ago
Stream listener should not be overwritten after triggering http-on-modify-request observers
Categories
(Core :: Networking: HTTP, defect, P5)
Core
Networking: HTTP
Tracking
()
NEW
People
(Reporter: jwkbugzilla, 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.
Updated•9 years ago
|
Whiteboard: [necko-backlog]
Comment 1•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Comment 2•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
Comment 3•4 years ago
|
||
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.
Description
•