2010-09-12 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Sep 2010 00:42:45 +0000 (00:42 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 13 Sep 2010 00:42:45 +0000 (00:42 +0000)
        Reviewed by Eric Seidel.

        isParsingFragment assert hit in new treebuilder
        https://bugs.webkit.org/show_bug.cgi?id=45621

        Add new tests and (minor) test progression.

        * html5lib/resources/webkit01.dat:
        * html5lib/runner-expected.txt:
2010-09-12  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        isParsingFragment assert hit in new treebuilder
        https://bugs.webkit.org/show_bug.cgi?id=45621

        Update our list of special tags to match the spec.

        * html/parser/HTMLTreeBuilder.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@67356 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/html5lib/resources/webkit01.dat
LayoutTests/html5lib/runner-expected.txt
WebCore/ChangeLog
WebCore/html/parser/HTMLTreeBuilder.cpp

index 2fb8883..a529cce 100644 (file)
@@ -1,3 +1,15 @@
+2010-09-12  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        isParsingFragment assert hit in new treebuilder
+        https://bugs.webkit.org/show_bug.cgi?id=45621
+
+        Add new tests and (minor) test progression.
+
+        * html5lib/resources/webkit01.dat:
+        * html5lib/runner-expected.txt:
+
 2010-09-12  Robert Hogan  <robert@webkit.org>
 
         Reviewed by Andreas Kling.
index c50ce10..7022813 100644 (file)
@@ -401,3 +401,34 @@ console.log("FOO<span>BAR</span>BAZ");
 |   " "
 |   <body>
 |     "ddd"
+
+#data
+<kbd><table></kbd><col><select><tr>
+#errors
+#document
+| <html>
+|   <head>
+|   <body>
+|     <kbd>
+|       <select>
+|       <table>
+|         <colgroup>
+|           <col>
+|         <tbody>
+|           <tr>
+
+#data
+<kbd><table></kbd><col><select><tr></table><div>
+#errors
+#document
+| <html>
+|   <head>
+|   <body>
+|     <kbd>
+|       <select>
+|       <table>
+|         <colgroup>
+|           <col>
+|         <tbody>
+|           <tr>
+|       <div>
index 974f6ca..6f4d487 100644 (file)
@@ -114,10 +114,10 @@ Got:
 |             <math math>
 |               <math mi>
 |                 <svg svg>
-|         <math mo>
-|     <span>
-|     <path>
-|     <mi>
+|               <math mo>
+|             <span>
+|             <path>
+|       <math mi>
 Expected:
 | <html>
 |   <head>
index 05912d0..fb66f95 100644 (file)
@@ -1,3 +1,14 @@
+2010-09-12  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        isParsingFragment assert hit in new treebuilder
+        https://bugs.webkit.org/show_bug.cgi?id=45621
+
+        Update our list of special tags to match the spec.
+
+        * html/parser/HTMLTreeBuilder.cpp:
+
 2010-09-12  Robert Hogan  <robert@webkit.org>
 
         Reviewed by Andreas Kling.
index c87d494..9241837 100644 (file)
@@ -132,11 +132,14 @@ bool isTableBodyContextTag(const AtomicString& tagName)
 // http://www.whatwg.org/specs/web-apps/current-work/multipage/parsing.html#special
 bool isSpecialNode(Node* node)
 {
+    if (node->hasTagName(SVGNames::foreignObjectTag))
+        return true;
     if (node->namespaceURI() != xhtmlNamespaceURI)
         return false;
-    // FIXME: This list is out of sync with the spec.
     const AtomicString& tagName = node->localName();
     return tagName == addressTag
+        || tagName == appletTag
+        || tagName == areaTag
         || tagName == articleTag
         || tagName == asideTag
         || tagName == baseTag
@@ -146,6 +149,7 @@ bool isSpecialNode(Node* node)
         || tagName == bodyTag
         || tagName == brTag
         || tagName == buttonTag
+        || tagName == captionTag
         || tagName == centerTag
         || tagName == colTag
         || tagName == colgroupTag
@@ -158,6 +162,7 @@ bool isSpecialNode(Node* node)
         || tagName == dtTag
         || tagName == embedTag
         || tagName == fieldsetTag
+        || tagName == figcaptionTag
         || tagName == figureTag
         || tagName == footerTag
         || tagName == formTag
@@ -176,12 +181,14 @@ bool isSpecialNode(Node* node)
         || tagName == liTag
         || tagName == linkTag
         || tagName == listingTag
+        || tagName == marqueeTag
         || tagName == menuTag
         || tagName == metaTag
         || tagName == navTag
         || tagName == noembedTag
         || tagName == noframesTag
         || tagName == noscriptTag
+        || tagName == objectTag
         || tagName == olTag
         || tagName == pTag
         || tagName == paramTag
@@ -191,8 +198,12 @@ bool isSpecialNode(Node* node)
         || tagName == sectionTag
         || tagName == selectTag
         || tagName == styleTag
+        || tagName == summaryTag
+        || tagName == tableTag
         || isTableBodyContextTag(tagName)
+        || tagName == tdTag
         || tagName == textareaTag
+        || tagName == thTag
         || tagName == titleTag
         || tagName == trTag
         || tagName == ulTag