Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-25 22:01:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 465921 - httpd.js fails to deliver after about 35 reloads of test_seek1.html, most likely because the connection is being prematurely terminated by the client. r=bz

Browse Source
This commit is contained in:
Jeff Walden 2008-12-15 21:02:40 -08:00
parent 630b09f732
commit 4a6ba4660c
1 changed files with 31 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
netwerk/test/httpserver/httpd.js
View File

@ -370,9 +370,7 @@ nsHttpServer.prototype =
*/
onSocketAccepted: function(socket, trans)
{
dumpn("*** onSocketAccepted(socket=" + socket + ", trans=" + trans + ") " +
"on thread " + gThreadManager.currentThread +
" (main is " + gThreadManager.mainThread + ")");
dumpn("*** onSocketAccepted(socket=" + socket + ", trans=" + trans + ")");
dumpn(">>> new connection on " + trans.host + ":" + trans.port);
@ -942,7 +940,13 @@ Connection.prototype =
this.server._handler.handleError(code, this, metadata);
},
/** Ends this connection, destroying the resources it uses. */
/**
* Ends this connection, destroying the resources it uses. This function
* should only be called after a response has been completely constructed,
* response headers have been sent, and the response body remains to be set,
* which all happens before ServerHandler._pendingRequests is incremented,
* because it handles the corresponding decrement of that value.
*/
end: function()
{
this.server._endConnection(this);
@ -1056,7 +1060,28 @@ RequestReader.prototype =
if (!data)
return;
data.appendBytes(readBytes(input, input.available()));
try
{
data.appendBytes(readBytes(input, input.available()));
}
catch (e)
{
if (e.result !== Cr.NS_ERROR_BASE_STREAM_CLOSED)
{
dumpn("*** WARNING: unexpected error when reading from socket; will " +
"be treated as if the input stream had been closed");
}
// We've lost a race -- input has been closed, but we're still expecting
// to read more data. available() will throw in this case, and since
// we're dead in the water now, destroy the connection. NB: we don't use
// end() here because that has interesting interactions with
// ServerHandler._pendingRequests.
dumpn("*** onInputStreamReady called on a closed input, destroying " +
"connection");
this._connection.destroy();
return;
}
switch (this._state)
{
@ -2630,6 +2655,7 @@ ServerHandler.prototype =
}
catch (e)
{
dumpn("*** error in handleError: " + e);
connection.close();
connection.server.stop();
}