Closed Bug 672958 Opened 13 years ago Closed 9 years ago

Have an API to explicitly set pipelining class on an HTTP channel, cleanup nsHttpTransaction::Classify()

Categories

(Core :: Networking: HTTP, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: mayhemer, Unassigned)

References

Details

There was added a classification of an HTTP request transaction in bug 599164 to group similar requests on a single connection and pipeline. The code that decides on the class [1] is not very clean, class is based on a vague check for Accept header value or very simple URL parsing. Consumers must be able to explicitly set the class on a channel using the new API. This API can also be used for much simpler pre-test implementation (bug 603505). My first suggestion is to add an attribute to nsIHttpChannelInternal called PipelineClass. Channel then propagates the class to the transaction. Image loader, script loader, css loader will then explicitly set the class. Default value will be GENERAL. Channel will have the right to reset to REVALIDATION, but only if class is not PRETEST. [1] http://hg.mozilla.org/projects/larch/file/f5e03f4cb36f/netwerk/protocol/http/nsHttpTransaction.cpp#l156
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.