479266 - Should support nested content encodings

Status: RESOLVED DUPLICATE of bug 717524

(Core :: Networking: HTTP, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

From RFC 2616 14.11:
   If multiple encodings have been applied to an entity, the content
   codings MUST be listed in the order in which they were applied.


We should support that. Currently our code looks like this:
735     const char *val = mResponseHead->PeekHeader(nsHttp::Content_Encoding);
736     if (gHttpHandler->IsAcceptableEncoding(val)) {
[...]
http://mxr.mozilla.org/mozilla-central/source/netwerk/protocol/http/src/nsHttpChannel.cpp#735

The code should split the header into its separate values and apply a sequence of stream converters.

This would be useful for the case of SDCH, which uses:
Content-Encoding: sdch, gzip

Comment 1

[Andrew Cook]

This has been fixed by https://bugzilla.mozilla.org/show_bug.cgi?id=717524
would mark this as a duplicate but i cannot seem to find the button,
or don't have permissions to, either way.

Comment 2

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

(In reply to Andrew from comment #1)
> This has been fixed by https://bugzilla.mozilla.org/show_bug.cgi?id=717524
> would mark this as a duplicate but i cannot seem to find the button,
> or don't have permissions to, either way.

Change status to "RESOLVED" and then choose "DUPLICATE" for the resolution.