Closed Bug 46225 Opened 24 years ago Closed 23 years ago

Mozilla ignores the HTTP mimetype of stylesheets.

Categories

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

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.7

People

(Reporter: ian, Assigned: bzbarsky)

Details

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

Attachments

(4 files, 2 obsolete files)

Stylesheet for test case, incorrectly labelled as text/html
(deleted), text/html
Details
Test case, which tries to use previous attachement as a stylesheet.
(deleted), text/html
Details
diff -uw of patch that only does this in strict mode and is case-insensitive
(deleted), patch
dbaron
: review+
Details | Diff | Splinter Review
diff -u of same patch
(deleted), patch
Details | Diff | Splinter Review 
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
Keywords: correctness, nsbeta3, testcase
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) (deleted) — — Splinter Review 

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Same patch as diff -uw for readability (obsolete) (deleted)
        — 
          — 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+

    
    

    
  


  

        Attachment #58233 -
        Attachment is obsolete: true

        Attachment #58234 -
        Attachment is obsolete: true

    
    

    
  

      
      
      
      

        Attached patch
          
          
        diff -u of same patch (deleted)
        — 
          — Splinter Review
      

    


  

    
    

    
  
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: 23 years ago
Resolution: --- → FIXED

    
    

    
  
verified
Status: RESOLVED → VERIFIED

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: