Bug #56934 --> properly determine the status for on end document load. We used to

use the status of the last request processed in loading the document to determine
 success or failure. That's incorrect. Instead, test to see if the load group is being
canceled. If it is, use that as the status for the entire document. Otherwise, ignore
the status for the last request and instead use the status for the main document
(the default load channel).

this also means making sure the http channel properly sets it's status code in ResponseComplete (r=gagan)
 sr=rpotts, r=sspitzer

netwerk/protocol/http/src/nsHTTPChannel.cpp

@@ -1917,6 +1917,7 @@ nsresult nsHTTPChannel::ResponseCompleted(nsIStreamListener *aListener,
     //
     // Call the consumer OnStopRequest(...) to end the request...
     //----------------------------------------------------------------
+    mStatus = aStatus; // set the channel's status based on the respone status
     if (aListener)
     {
         rv = aListener->OnStopRequest(this, mResponseContext, aStatus, aStatusArg);