Closed Bug 109123 Opened 23 years ago Closed 23 years ago

"-//IETF//DTD HTML 3//EN" should trigger quirks mode

Categories

(Core :: DOM: HTML Parser, defect, P1)

Product:
Core
Component:
DOM: HTML Parser
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.7

People

(Reporter: vlad, Assigned: dbaron)

References

(
URL
)

Details

Attachments

(1 file, 1 obsolete file)

patch
1.14 KB, patch
harishd
: review+
jst
: superreview+
Details | Diff | Splinter Review 
The URL creates 3 frames; only the top navigation frame has content displayed,
whereas the left "table of contents" view and the right "content view" are
empty, even though viewing the source on those frames displays content.  Opening
the frames in a separate window doesn't help.

Always reproducible.  0.9.5-ximian.1 build (0.9.5 release source).
The page has broken comment syntax (an extraneous "--" in the comment) and the
following doctype:

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 3//EN">

We treat the page in strict mode.

David, should we do strict mode for that doctype?  If so, punt to evangelism.
Assignee: asa → dbaron
Status: UNCONFIRMED → NEW
Component: Browser-General → Parser
Ever confirmed: true
OS: Linux → All
Hardware: PC → All
Argh, I thought it might be something like that.  I'll submit a bug to IBM as
well -- all of their DB2 online docs have this problem (about 40MB of html).
No, it's our bug -- we should handle that as quirks mode.
Status: NEW → ASSIGNED
Priority: -- → P1
Summary: no content is displayed in frames in URL (view source shows content present) → "-//IETF//DTD HTML 3//EN" should trigger quirks mode
Target Milestone: --- → mozilla0.9.7
*** Bug 110178 has been marked as a duplicate of this bug. ***
Same thing for:

<!doctype html public "-//IETF//DTD HTML 3.2 Final//EN">
*** Bug 111495 has been marked as a duplicate of this bug. ***
Attached patch Adds the 2 doctypes (obsolete) — — Splinter Review 

    
    

    
  
Those are out of order, right (so that it would assert in DEBUG builds since the
binary search would fail)?  Space (0x20) comes before slash (0x2F).

    
    

    
  

      
      
      
      

        Attached patch
          
          
        patch
        — 
          — Splinter Review
      

    


  

        Attachment #60022 -
        Attachment is obsolete: true

    
    

    
  
Comment on attachment 60845 [details] [diff] [review]
patch

r=harishd

        Attachment #60845 -
        Flags: review+

    
    

    
  
Comment on attachment 60845 [details] [diff] [review]
patch

sr=jst

        Attachment #60845 -
        Flags: superreview+

    
    

    
  
Fix checked in 2001-12-08 14:47 PDT.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED

    
    

    
  
verified fixed.
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: