580451 - Document PHttpChannel.ipdl

Status: RESOLVED DUPLICATE of bug 576692

(Core :: Networking: HTTP, defect)

Description

[Josh Matthews [:jdm]]

>+++ b/netwerk/protocol/http/PHttpChannel.ipdl
>+  AsyncOpen(URI                 uri,
>+            // - TODO: bug 571161: unclear if any HTTP channel clients ever
>+            // set originalURI != uri (about:credits?); also not clear if
>+            // chrome channel would ever need to know.  Get rid of next arg?
>+            URI                 original,
>+            URI                 doc,

Probably worth documenting at least the |doc| argument.

>+            PRInt32             uploadStreamInfo,

Document?  That's pretty opaque.

>+  OnStartRequest(nsHttpResponseHead responseHead);
>+
>+  OnDataAvailable(nsCString data, 
>+                  PRUint32  offset, 
>+                  PRUint32  count);
>+
>+  OnStopRequest(nsresult statusCode);

Document that these follow the usual nsIChannel contract (if they do) or what
contract they follow otherwise?