Closed Bug 502984 Opened 15 years ago Closed 15 years ago

[HTML5] CDATA escapes need to close after --\s*!>

Categories

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

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

Tracking

()

Status:
RESOLVED DUPLICATE of bug 503632

People

(Reporter: fullmetaljacket.xp+bugmail, Unassigned)

References

(
URL
)

Details

(Keywords: testcase)

Attachments

(1 file, 1 obsolete file)

Test case
134 bytes, text/html
Details 
http://www.erc.gov.ph/

the url above does not load (empty white page) when html5 parser is turned on.
Fishy comments in the script

From the source code of that page:

<script language="JavaScript">

<!-- hide script from old browsers
// Script to create a drop-down menu effect using layers.
//(c)Copyright Daren Craddock 2002
//You may use this code in any application, no limits,
//provided that you acknowledge Daren Craddock as the originator.
// big() makes selected layer longer (height property)

removing the first line allows the page to load.
The problem is -- !>. Need to escalate this as a spec issue.
Already landed. Not blocking the landing.
No longer blocks: html5-parsing-land
Summary: [HTML5] website does not load when html5 parser is enabled → [HTML5] CDATA escapes need to close after --!\s*>
Spec bug:
http://www.w3.org/Bugs/Public/show_bug.cgi?id=7089
Summary: [HTML5] CDATA escapes need to close after --!\s*> → [HTML5] CDATA escapes need to close after --\s*!>
OS: Windows 7 → All
Hardware: x86 → All
Need a regression test here too.
Flags: in-testsuite?
Blocks: html5-parsing
Attached file testcase (obsolete) — 

    
  
Keywords: testcase

    
    

    
  
Note for anyone searching for duplicates:
If you've seen this in the wild elsewhere, please note the URL here.

    
    

    
  
Comment on attachment 389404 [details]
testcase

The test case tests comments rather than CDATA escapes.

        Attachment #389404 -
        Attachment is obsolete: true

    
    

    
  

      
      
      
      

        Attached file
          
        Test case
        — 
      

    


  
Priority: -- → P1

    
    

    
  
Fixed by the fix for bug 503632 even though not strictly a dupe.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE

    
    

    
  
verified fixed by bug 503632 using testcase posted

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: