Changeset 21428 in webkit

Timestamp:

May 12, 2007, 2:15:00 PM (18 years ago)

Author:

hyatt

Message:

Fix for bug 12740, bmw.com doesn't work. We were missing a rule that is covered by the HTML5
parsing spec. A <table> encountered while inside stray table content should close up the
table responsible for the stray content and start the new table as a sibling of the first.

Reviewed by mjs

fast/invalid/table-inside-stray-table-content.html

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLParser.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2007-05-12  David Hyatt  <hyatt@apple.com>
+
+        Fix for bug 12740, bmw.com doesn't work.  We were missing a rule that is covered by the HTML5
+        parsing spec.  A <table> encountered while inside stray table content should close up the
+        table responsible for the stray content and start the new table as a sibling of the first.
+
+        Reviewed by mjs
+
+        fast/invalid/table-inside-stray-table-content.html
+
 2007-05-12  Eric Seidel  <eric@webkit.org>
 

trunk/WebCore/html/HTMLParser.cpp

@@ -282 +282 @@
     const AtomicString& localName = n->localName();
     int tagPriority = n->isHTMLElement() ? static_cast<HTMLElement*>(n)->tagPriority() : 0;
-        
+    
+    // <table> is never allowed inside stray table content.  Always pop out of the stray table content
+    // and close up the first table, and then start the second table as a sibling.
+    if (inStrayTableContent && localName == tableTag)
+        popBlock(tableTag);
+    
     // let's be stupid and just try to insert it.
     // this should work if the document is well-formed