[Webkit-unassigned] [Bug 4648] Shockwave unable to load GZip'd text resources when server sends Content-Length header

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Tue Mar 7 20:56:08 PST 2006


http://bugzilla.opendarwin.org/show_bug.cgi?id=4648


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6922|review?                     |review-
               Flag|                            |




------- Comment #6 from darin at apple.com  2006-03-07 20:56 PDT -------
(From update of attachment 6922)
This patch is not quite right. The content encoding might be "identity", so
just the presence of a content encoding header field is not the correct check.
Also, buggy servers sometimes send strings like "identity, identity".

It's hard to code this correctly at the WebKit level and stay in sync. with the
features of the NSURL level; ideally we would have a Foundation method like
expectedContentLength, but one that knew to only give us the length for things
it won't decode. In general we don't want code that has to second-guess what
NSURL is doing for us.

Current versions of NSURLConnection support "deflate" and "gzip"; anything else
is treated as "identity". The comparison is done in a case insensitive manner.
NSURLConnection knows how to parse off an optional "x-" prefix. It knows how to
ignore parameters that come after an optional ";" separator. It knows how to
ignore whitespace. It parses a list separated by commas; if all the items in
the list are the same (ignoring case, whitespace, "x-" prefixes, and
parameters) then it uses it, otherwise it treats it as "identity".

Hard-coding this precise rule into WebBaseNetscapePluginStream would be kinda
lame, but I don't have a better idea.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list