Closed Bug 94269 Opened 23 years ago Closed 23 years ago

Unclosed <p> before <form> results in wrong document tree

Categories

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

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

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.7

People

(Reporter: hsivonen, Assigned: harishd)

References

(
URL
)

Details

(Keywords: testcase, Whiteboard: [fix in hand])

Attachments

(4 files)

Test case
380 bytes, text/html
Details
Actual results
13.16 KB, image/jpeg
Details
Expected results
11.77 KB, image/jpeg
Details
patch v1.1 [ P should not contain FORM ]
862 bytes, patch
hjtoi-bugzilla
: review+
jst
: superreview+
Details | Diff | Splinter Review 
Build ID: 2001080105 FizzillaCFM (trunk)

Reproducible: Always.

Steps to reproduce:
1) Load the attached test case.

Actual results:
The <form> element is considered to be inside the unclosed <p> and is closed
prematurely when the next <p> occurs.
Expected results:
Since <form> is a block-level element, expected it to close <p> implicitly.
Attached file Test case — 

    
    

    
  

      
      
      
      

        Attached image
          
        Actual results
        — 
      

    


  

    
    

    
  

      
      
      
      

        Attached image
          
        Expected results
        — 
      

    


  

    
    

    
  
Dupe of 91927?  But that's only for strict mode...

    
    

    
  
Setting to m0.9.5
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.5

    
    

    
  
Surely not Mac OS X specific.
OS: MacOS X → All
Hardware: Macintosh → All

    
  
Priority: -- → P3

    
  
Blocks: html4.01

    
  
QA Contact: bsharma → moied

    
    

    
  
--> 0.9.6
Target Milestone: mozilla0.9.5 → mozilla0.9.6

    
    

    
  
Out of time :-( Moving to 0.9.7
Target Milestone: mozilla0.9.6 → mozilla0.9.7

    
  
Whiteboard: [fix in hand]
Keywords: patch, 
          
            testcase

    
    

    
  
Comment on attachment 57838 [details] [diff] [review]
patch v1.1 [ P should not contain FORM ]

sr=jst

        Attachment #57838 -
        Flags: superreview+

    
    

    
  
Comment on attachment 57838 [details] [diff] [review]
patch v1.1 [ P should not contain FORM ]

r=heikki

        Attachment #57838 -
        Flags: review+

    
    

    
  
*** Bug 108574 has been marked as a duplicate of this bug. ***

    
    

    
  
FIXED.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED

    
    

    
  
Verified fixed checked in cvs Version 3.135



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: