[Webkit-unassigned] [Bug 25352] REGRESSION (r42322): style isn't applied at bmwusa.com due to Content-Type with comma in it

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=25352





------- Comment #10 from abarth at webkit.org  2009-05-02 12:43 PDT -------
Looks like the work is done at
http://mxr.mozilla.org/mozilla-central/source/netwerk/base/src/nsURLHelper.cpp#869.
 That code has the following BNF as a comment:

878     // Augmented BNF (from RFC 2616 section 3.7):
879     //
880     //   header-value = media-type *( LWS "," LWS media-type )
881     //   media-type   = type "/" subtype *( LWS ";" LWS parameter )
882     //   type         = token
883     //   subtype      = token
884     //   parameter    = attribute "=" value
885     //   attribute    = token
886     //   value        = token | quoted-string
887     //   
888     //
889     // Examples:
890     //
891     //   text/html
892     //   text/html, text/html
893     //   text/html,text/html; charset=ISO-8859-1
894     //   text/html,text/html; charset="ISO-8859-1"
895     //   text/html;charset=ISO-8859-1, text/html
896     //   text/html;charset='ISO-8859-1', text/html
897     //   application/octet-stream

The code doesn't quite seem to follow this BNF.  For example,
http://mxr.mozilla.org/mozilla-central/source/netwerk/base/src/nsURLHelper.cpp#766
treats the "(" character as special.  Also, media types of "*/*" and those
without a "/" character are treated like the empty media type:
http://mxr.mozilla.org/mozilla-central/source/netwerk/base/src/nsURLHelper.cpp#823


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