Remove SSL thread (nsSSLThread) hacks implemented for SPDY

RESOLVED FIXED

Status

()

Core
Security: PSM
--
enhancement
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: briansmith, Unassigned)

Tracking

({perf})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Patrick's SPDY patches include some un-pretty (but working) changes to nsSSLThread and related code required to get SPDY working in the absense of a fix for bug 674147. Once bug 674147 is fixed, most of the ugliness will automatically be removed via the removal of the nsSSLThread.cpp and nsSSLThread.h files, but there might still be some additional cleanup required.
cleaned up the poll() hack in transport service and the use of nshttpconnection::onsocketreadable() to drive npn, as well as the use of the sslthread for forcehandshake().

That clears up everything I considered a sslthread workaround.

I'm going to mark this fixed even though I can't point to a checkin (because the hacks were never checked in)
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.