While working with bryner on one of the bfcache bugs, we found that nsIRequest::isPending is not set to false in a uniform way. For example, the HTTP channel sets isPending to false before calling OnStopRequest, while other channels set isPending to false after calling OnStopRequest. It would be nice if all channels set IsPending to false before calling OnStopRequest so that way downstream someone could determine if the channel is in the process of calling OnStopRequest. for example, in the case of a multipart mixed response, it would have been nice if one could inspect the IsPending state of the partChannel's baseChannel to determine if the OnStopRequest being called is the last OnStopRequest. bryner added nsIMultiPartChannel::IsLastPart to get around this problem. there may be other cases where being consistent here matters, so we should fix it.
Assignee: darin.moz → nobody
QA Contact: benc → networking
I think we should fix this for 1.9...
biesi, are you interested in taking this?
Flags: blocking1.9? → blocking1.9-
11 years ago
Assignee: nobody → cbiesinger
6 years ago
Assignee: cbiesinger → nobody
Honza, Michal, Patrick - do you guys know if this is still an issue?
I don't know if there's inconsistency between protocols anymore, but we're still inconsitstent *within* HTTP: see bug 748117
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P2 → P5
You need to log in before you can comment on or make changes to this bug.