Changeset 21428 in webkit


Ignore:
Timestamp:
May 12, 2007 2:15:00 PM (17 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:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r21427 r21428  
     12007-05-12  David Hyatt  <hyatt@apple.com>
     2
     3        Fix for bug 12740, bmw.com doesn't work.  We were missing a rule that is covered by the HTML5
     4        parsing spec.  A <table> encountered while inside stray table content should close up the
     5        table responsible for the stray content and start the new table as a sibling of the first.
     6
     7        Reviewed by mjs
     8
     9        fast/invalid/table-inside-stray-table-content.html
     10
    1112007-05-12  Eric Seidel  <eric@webkit.org>
    212
  • trunk/WebCore/html/HTMLParser.cpp

    r21208 r21428  
    282282    const AtomicString& localName = n->localName();
    283283    int tagPriority = n->isHTMLElement() ? static_cast<HTMLElement*>(n)->tagPriority() : 0;
    284        
     284   
     285    // <table> is never allowed inside stray table content.  Always pop out of the stray table content
     286    // and close up the first table, and then start the second table as a sibling.
     287    if (inStrayTableContent && localName == tableTag)
     288        popBlock(tableTag);
     289   
    285290    // let's be stupid and just try to insert it.
    286291    // this should work if the document is well-formed
Note: See TracChangeset for help on using the changeset viewer.