2010-05-31 Adam Barth <abarth@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Jun 2010 07:06:35 +0000 (07:06 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Jun 2010 07:06:35 +0000 (07:06 +0000)
        Reviewed by Eric Seidel.

        Fix default action for EndTagOpenState
        https://bugs.webkit.org/show_bug.cgi?id=39982

        Add a test for a bogus end tag to webkit01.dat and update expected
        results now that we pass some more tests.

        Adding this test revealed a bug in resuming the bogus comment state.
        I've left these broken expectations, but I'll fix the bug in a future
        patch. (The bug existed prior to this patch, just not the test.)

        * html5lib/resources/webkit01.dat:
        * html5lib/runner-expected-html5.txt:
        * html5lib/webkit-resumer-expected-html5.txt:
2010-05-31  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        Fix default action for EndTagOpenState
        https://bugs.webkit.org/show_bug.cgi?id=39982

        Add a test for a bogus end tag to webkit01.dat and update expected
        results now that we pass some more tests.

        Adding this test revealed a bug in resuming the bogus comment state.
        I've left these broken expectations, but I'll fix the bug in a future
        patch. (The bug existed prior to this patch, just not the test.)

        * html5lib/resources/webkit01.dat:
        * html5lib/runner-expected-html5.txt:
        * html5lib/webkit-resumer-expected-html5.txt:

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

LayoutTests/ChangeLog
LayoutTests/html5lib/resources/webkit01.dat
LayoutTests/html5lib/runner-expected-html5.txt
LayoutTests/html5lib/runner-expected.txt
LayoutTests/html5lib/webkit-resumer-expected-html5.txt
LayoutTests/html5lib/webkit-resumer-expected.txt
WebCore/ChangeLog
WebCore/html/HTML5Lexer.cpp

index 6614b89..31f08db 100644 (file)
@@ -1,3 +1,21 @@
+2010-05-31  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Fix default action for EndTagOpenState
+        https://bugs.webkit.org/show_bug.cgi?id=39982
+
+        Add a test for a bogus end tag to webkit01.dat and update expected
+        results now that we pass some more tests.
+
+        Adding this test revealed a bug in resuming the bogus comment state.
+        I've left these broken expectations, but I'll fix the bug in a future
+        patch. (The bug existed prior to this patch, just not the test.)
+
+        * html5lib/resources/webkit01.dat:
+        * html5lib/runner-expected-html5.txt:
+        * html5lib/webkit-resumer-expected-html5.txt:
+
 2010-05-31  Roland Steiner  <rolandsteiner@chromium.org>
 
         Unreviewed: land new Chromium baselines to unblock WebKit roll
index f2cb7d3..5adf1bb 100644 (file)
@@ -156,3 +156,12 @@ console.log("FOO<span>BAR</span>BAZ");
 |     <script>
 |       "document.write('3')"
 |     "34"
+
+#data
+</ tttt>
+#errors
+#document
+| <!--  tttt -->
+| <html>
+|   <head>
+|   <body>
index a8b9d12..9f55890 100644 (file)
@@ -397,7 +397,8 @@ resources/tests16.dat:
 184
 186
 
-resources/webkit01.dat: PASS
+resources/webkit01.dat:
+13
 
 resources/doctype01.dat:
 2
index e832cef..fa83b8e 100644 (file)
@@ -157,5 +157,10 @@ CONSOLE MESSAGE: line 2: FOO<span>BAR</span>BAZ
 CONSOLE MESSAGE: line 2: FOO<span>BAR</span>BAZ
 CONSOLE MESSAGE: line 2: FOO<span>BAR</span>BAZ
 Content-Type: text/plain
-resources/webkit01.dat: PASS
+resources/webkit01.dat:
+519.3
+520.4
+521.5
+522.6
+523.7
 #EOF
index c6c31a0..46dc713 100644 (file)
@@ -156,4 +156,11 @@ CONSOLE MESSAGE: line 3: FOO<span>BAR</span>BAZ
 CONSOLE MESSAGE: line 3: FOO<span>BAR</span>BAZ
 CONSOLE MESSAGE: line 3: FOO<span>BAR</span>BAZ
 CONSOLE MESSAGE: line 3: FOO<span>BAR</span>BAZ
-resources/webkit01.dat: PASS
+resources/webkit01.dat:
+517.1
+518.2
+519.3
+520.4
+521.5
+522.6
+523.7
index 28a1f20..297d807 100644 (file)
@@ -1,3 +1,21 @@
+2010-05-31  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        Fix default action for EndTagOpenState
+        https://bugs.webkit.org/show_bug.cgi?id=39982
+
+        Add a test for a bogus end tag to webkit01.dat and update expected
+        results now that we pass some more tests.
+
+        Adding this test revealed a bug in resuming the bogus comment state.
+        I've left these broken expectations, but I'll fix the bug in a future
+        patch. (The bug existed prior to this patch, just not the test.)
+
+        * html5lib/resources/webkit01.dat:
+        * html5lib/runner-expected-html5.txt:
+        * html5lib/webkit-resumer-expected-html5.txt:
+
 2010-05-31  Lyon Chen  <liachen@rim.com>
 
         Reviewed by Kent Tamura.
index 098ae53..6c23bb0 100644 (file)
@@ -436,7 +436,8 @@ bool HTML5Lexer::nextToken(SegmentedString& source, HTML5Token& token)
                 m_state = DataState;
             } else {
                 emitParseError();
-                m_state = DataState;
+                m_state = BogusCommentState;
+                continue;
             }
             // FIXME: Handle EOF properly.
             break;