Closed Bug 2451 Opened 26 years ago Closed 26 years ago

Conn: Failure to handle PowerBook sleep (NSPR OT notification)

Categories

(Core :: Networking, defect, P2)

PowerPC
All
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: gordon)

Details

(Keywords: testcase)

The current viewerDebug has a nasty surprise waiting for anyone who should attempt to put a laptop to sleep while running the viewer. The good news is that the viewer is stable enough not to crash between launch and the time when one might want to put the laptop to sleep now. The bad news is that when attempting to put my PB G3 to sleep gives me a dprintf in Assertion Failure in macsockopt.c, line 247. It looks like the notifier routine: pascal void NotifierRoutine(void * contextPtr, OTEventCode code, OTResult result, void * cookie) isn't handling the OTEventCode that corresponds to dropping the network connection when putting the laptop to sleep. At worst macsockopt should probably ignore the message. Not sure what the "right" thing to do here is, but dropping into the debugger with the screen closed probably isn't the right thing.
Status: NEW → ASSIGNED
OS: All
This will probably get fixed as a side effect of fixing bug #2133. If not, I will deal with it explicitly.
Setting all current Open/Normal to M4.
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Gordon, is this a netlib or an NSPR bug?
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I've checked in fixes for #2133, which fix this as well. One of the changes removed the PR_ASSERT() from the NotifierRoutine for the sleep event (among others).
QA Contact: 3819 → 4082
Hey, Greg, can you verify this for me? I hear you have a PowerBook. Thanks.
Status: RESOLVED → VERIFIED
I don't have a debug build, but I can say with optimized for a while now my G3 PBook has handled sleep and Seamonkey without trouble. Marking verified with apprunner in March 23 builds.
Will have saari@netscape.com check out a debug build..leaving verified and will reopen if necessary.
verified fixed with viewer debug on saari's G3 laptop.
Changing all Networking Library/Browser bugs to Networking-Core component for Browser. Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do this in a bulk change. If this happens, I will fix. ;-)
Bulk move of all Networking-Core (to be deleted component) bugs to new Networking component.
Keywords: testcase
Summary: Failure to handle PowerBook sleep (NSPR OT notification) → Conn: Failure to handle PowerBook sleep (NSPR OT notification)
You need to log in before you can comment on or make changes to this bug.