nsInputStreamChannel is smart and if it's already canceled will ignore Cancel() calls. You could make a similar change on 1.8 branch, I suppose, so you don't have to copy this SyncChannelStatus thing. Or you could just Cancel() if rv is a failure and not worry about clobbering an existing error status if there is one. But we do need to do something, both trunk and branch.
(In reply to comment #0) > nsInputStreamChannel is smart and if it's already canceled will > ignore Cancel() calls. On trunk, that is.
You need to log in before you can comment on or make changes to this bug.