twitter-archive / CocoaSPDY

SPDY for iOS and OS X
Apache License 2.0
2.39k stars 233 forks source link

Handle some stream delegate edge cases better. #75

Closed kgoodier closed 9 years ago

kgoodier commented 9 years ago

The NSAssert in SPDYSessionManager in streamCanceled has been seen to fire, so this change attempts to address those by very explicitly managing the stream's delegate pointer when ownership of the stream changes.

Also includes some unit tests covering these cases. They ended up requiring a lot of set up code and internal knowledge, but it's good enough for now.