46225 - Mozilla ignores the HTTP mimetype of stylesheets.

Status: VERIFIED FIXED

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

Description

[Hixie (not reading bugmail)]

STEPS TO REPRODUCE:
   Open the attachement.

ACTUAL RESULTS:
   Stylesheet, which is of type text/html, is treated as text/css.

EXPECTED RESULTS:
   Stylesheet should be ignored as we do not support text/html stylesheets.

Comment 1

[Hixie (not reading bugmail)]

Attachment: Stylesheet for test case, incorrectly labelled as text/html

Comment 2

[Hixie (not reading bugmail)]

Attachment: Test case, which tries to use previous attachement as a stylesheet.

Comment 3

[Hixie (not reading bugmail)]

TESTED ON:
   Mozilla 2000072120 Windows 2000
   Commericial 2000072120 Windows 2000

This is a violation of the HTTP spec. (The 'type' attribute in HTML is only
advisory, not authorative.) Nominating for nsbeta3.

Comment 4

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

I think this is the same as bug 18817, which was marked fixed in May.

Comment 5

[ekrock's old account (dead)]

[nsbeta3-] and FUTURE.

Comment 6

[Hixie (not reading bugmail)]

Taking QA per managerial policy.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch to fix this

Comment 8

[Boris Zbarsky [:bzbarsky]]

Attachment: Same patch as diff -uw for readability

Comment 9

[Boris Zbarsky [:bzbarsky]]

Taking this....

Comment 10

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

Comment on attachment 58234 [details] [diff] [review]
Same patch as diff -uw for readability

r=dbaron

Comment 11

[Pierre Saslawsky]

dbaron: Nav4 and IE ignore the mimetype.  I think that we should check it in 
strict mode only as you recommended in bug 18817 comment #4.

The fix for bug 18817 is at http://lxr.mozilla.org/seamonkey/source/content/html/
document/src/nsHTMLContentSink.cpp#4436

Comment 12

[Marc Attinasi]

Comment on attachment 58234 [details] [diff] [review]
Same patch as diff -uw for readability

sr=attinasi | minor question though: does the case of "text/css" matter here?

Comment 13

[Boris Zbarsky [:bzbarsky]]

Attachment: diff -uw of patch that only does this in strict mode and is case-insensitive

Comment 14

[Boris Zbarsky [:bzbarsky]]

Attachment: diff -u of same patch

Comment 15

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

Comment on attachment 58385 [details] [diff] [review]
diff -uw of patch that only does this in strict mode and is case-insensitive

r=dbaron

Comment 16

[Boris Zbarsky [:bzbarsky]]

attachment 58387 [details] [diff] [review] checked in

Comment 17

[Madhur Bhatia]

verified