161268 - [FIX]Content-Style-Type ignored in various situations.

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P2)

Description

[Jim Ley]

Using 
<meta http-equiv="Content-Style-Type" content="text/x-jl-css" />
to set the style language to my style language, Mozilla correctly ignores it if 
the document is html served as text/html, if it's xml served as text/xml with 
meta and b elements from the xhtml namespace, it incorrectly applies the styles 
as if they were text/css (see the URL).

Equally having a Content-Style-Type HTTP header itself is completely ignored 
everywhere it seems to me.

http://www.w3.org/TR/REC-html40/present/styles.html  for the relevant bit of 
the standard.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: This should fix <meta> for XML and actual header for XML and HTML

Comment 2

[Boris Zbarsky [:bzbarsky]]

Jim, any chance of a testcase that uses HTTP headers so I can test that part of
the patch?

Comment 3

[Jim Ley]

Couple more test-cases:

XHTML 1.0 served as text/xml with content-style-type header:

http://jibbering.com/2002/8/test.2.1

HTML 4.01 served as text/html with content-style-type header:

http://jibbering.com/2002/8/test.1

Comment 4

[Boris Zbarsky [:bzbarsky]]

Perfect. Patch fixes both, by the way.  ;)

Comment 5

[Jim Ley]

Great, so that's 43 minutes from report to fix... I guess that's reasonable :-)

Comment 6

[Peter Van der Beken [:peterv]]

Comment on attachment 94160 [details] [diff] [review]
This should fix <meta> for XML and actual header for XML and HTML

r=peterv.

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I thought the HTTP header part of the bug was a duplicate of a very old bug Ian
filed, but I can't seem to find it.  I suspect Ian has some testcases in
http://www.bath.ac.uk/~py8ieh/internet/importtest/

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 94160 [details] [diff] [review]
This should fix <meta> for XML and actual header for XML and HTML

sr=dbaron

Comment 9

[Jim Ley]

Another Test case from hixie:

http://www.bath.ac.uk/~py8ieh/internet/importtest/extra/defaultlanguage.http.htm
l

(this one nicely telling you what it should look like etc...)

Comment 10

[Boris Zbarsky [:bzbarsky]]

Excellent.  Passes that too.  ;)

Fix checked in.

Comment 11

[Madhur Bhatia]

verified