Closed Bug 46225 Opened 19 years ago Closed 18 years ago

Mozilla ignores the HTTP mimetype of stylesheets.

Categories

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

defect

Tracking

()

VERIFIED FIXED
mozilla0.9.7

People

(Reporter: ian, Assigned: bzbarsky)

Details

(Keywords: testcase, Whiteboard: [Hixie-P4] [nsbeta3-])

Attachments

(4 files, 2 obsolete files)

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.
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.
Assignee: py8ieh=bugzilla → pierre
I think this is the same as bug 18817, which was marked fixed in May.
[nsbeta3-] and FUTURE.
Whiteboard: [nsbeta3-]
Target Milestone: --- → Future
Taking QA per managerial policy.
QA Contact: chrisd → py8ieh=bugzilla
Whiteboard: [nsbeta3-] → [Hixie-P4] [nsbeta3-]
Attached patch Patch to fix this (obsolete) — Splinter Review
Taking this....
Assignee: pierre → bzbarsky
Keywords: patch, review
OS: Windows 2000 → All
Priority: P3 → P2
Hardware: PC → All
Target Milestone: Future → mozilla0.9.7
Comment on attachment 58234 [details] [diff] [review]
Same patch as diff -uw for readability

r=dbaron
Attachment #58234 - Flags: review+
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 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?
Attachment #58234 - Flags: superreview+
Comment on attachment 58385 [details] [diff] [review]
diff -uw of patch that only does this in strict mode and is case-insensitive

r=dbaron
Attachment #58385 - Flags: review+
attachment 58387 [details] [diff] [review] checked in
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.