Closed Bug 148192 Opened 22 years ago Closed 16 years ago

Remove session-logout code from networking

Categories

(Core :: Networking, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.9.1a2

People

(Reporter: morse, Assigned: sgautherie)

References

Details

Attachments

(1 file)

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
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")) {
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!
Severity: normal → trivial
Target Milestone: --- → Future
We'd also remove this from mailnews backend code (SM and Tbird) as well, if OS/2
stops needing it.
-> default owner
Assignee: darin → nobody
Component: Networking: HTTP → Networking
QA Contact: tever → networking
Target Milestone: Future → ---
Depends on: 98673
OS: Windows NT → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1a1
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #325992 - Flags: superreview?(cbiesinger)
Attachment #325992 - Flags: review?(cbiesinger)
Attachment #325992 - Flags: superreview?(cbiesinger)
Attachment #325992 - Flags: superreview+
Attachment #325992 - Flags: review?(cbiesinger)
Attachment #325992 - Flags: review+
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
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: mozilla1.9.1a1 → mozilla1.9.1a2
[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
Please don't verify your own bugs.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: