Closed
Bug 148192
Opened 22 years ago
Closed 16 years ago
Remove session-logout code from networking
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla1.9.1a2
People
(Reporter: morse, Assigned: sgautherie)
References
Details
Attachments
(1 file)
2.06 KB,
patch
|
Biesinger
:
review+
Biesinger
:
superreview+
|
Details | Diff | Splinter Review |
As a result of the patch in bug 98673, the session-logout signal is no longer being issued. The code for handling that signal should be removed. This occurs in the following places in networking
Reporter | ||
Comment 1•22 years ago
|
||
And the places are (I forgot to put it into previous comment) /netwerk/protocol/http/src/nsHttpHandler.cpp, line 264 -- observerSvc->AddObserver(this, "session-logout", PR_TRUE); /netwerk/protocol/http/src/nsHttpHandler.cpp, line 2087 -- !nsCRT::strcmp(topic, "session-logout")) {
Comment 2•22 years ago
|
||
mkaply: you mentioned wanting to move OS/2 over the windows way of quick launch, right? if not, then i should mark this bug WONTFIX. please let me know what your plans are. thx!
Updated•19 years ago
|
Severity: normal → trivial
Target Milestone: --- → Future
Comment 3•19 years ago
|
||
We'd also remove this from mailnews backend code (SM and Tbird) as well, if OS/2 stops needing it.
Comment 4•18 years ago
|
||
-> default owner
Assignee: darin → nobody
Component: Networking: HTTP → Networking
QA Contact: tever → networking
Target Milestone: Future → ---
Assignee | ||
Updated•16 years ago
|
OS: Windows NT → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1a1
Assignee | ||
Comment 5•16 years ago
|
||
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #325992 -
Flags: superreview?(cbiesinger)
Attachment #325992 -
Flags: review?(cbiesinger)
Updated•16 years ago
|
Attachment #325992 -
Flags: superreview?(cbiesinger)
Attachment #325992 -
Flags: superreview+
Attachment #325992 -
Flags: review?(cbiesinger)
Attachment #325992 -
Flags: review+
Assignee | ||
Updated•16 years ago
|
Keywords: checkin-needed
Comment on attachment 325992 [details] [diff] [review] (Av1) <nsHttpHandler.cpp> (Checkin: Comment 6) http://hg.mozilla.org/mozilla-central/index.cgi/rev/5b3fc4a54cb1
Attachment #325992 -
Attachment description: (Av1) <nsHttpHandler.cpp> → (Av1) <nsHttpHandler.cpp> (Checkin: Comment 6)
Keywords: checkin-needed
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: mozilla1.9.1a1 → mozilla1.9.1a2
Assignee | ||
Comment 7•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008080208 SeaMonkey/2.0a1pre] (home, debug) (W2Ksp4) V.Fixed
Status: RESOLVED → VERIFIED
Comment 8•16 years ago
|
||
Please don't verify your own bugs.
Updated•16 years ago
|
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Updated•16 years ago
|
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•