2010-08-24 Dumitru Daniliuc <dumi@chromium.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Aug 2010 05:35:19 +0000 (05:35 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Aug 2010 05:35:19 +0000 (05:35 +0000)
        Unreviewed, Chromium's LayoutTestController does not implement markerTextForListItem() yet.

        * platform/chromium/test_expectations.txt:
2010-08-24  Eric Seidel  <eric@webkit.org>

        Reviewed by Adam Barth.

        Fix one more html5lib test case for button scoping
        https://bugs.webkit.org/show_bug.cgi?id=44583

        I <3 test driven development.

        * html/HTMLTreeBuilder.cpp:
        (WebCore::HTMLTreeBuilder::processStartTagForInBody):

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

LayoutTests/ChangeLog
LayoutTests/html5lib/runner-expected.txt
WebCore/ChangeLog
WebCore/html/HTMLTreeBuilder.cpp

index 5227136..2aa5340 100644 (file)
@@ -8,6 +8,15 @@
 
         Reviewed by Adam Barth.
 
+        Fix one more html5lib test case for button scoping
+        https://bugs.webkit.org/show_bug.cgi?id=44583
+
+        * html5lib/runner-expected.txt:
+
+2010-08-24  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Barth.
+
         Remove HTML5 parser testing infrastructure now that we don't need it
         https://bugs.webkit.org/show_bug.cgi?id=44581
 
index 63885dc..5979114 100644 (file)
@@ -87,28 +87,8 @@ Expected:
 |   <head>
 |   <body>
 |     <keygen>
-resources/tests20.dat:
-16
+resources/tests20.dat: PASS
 
-Test 16 of 25 in resources/tests20.dat failed. Input:
-<!doctype html><p><button><table>
-Got:
-| <!DOCTYPE html>
-| <html>
-|   <head>
-|   <body>
-|     <p>
-|       <button>
-|         <p>
-|         <table>
-Expected:
-| <!DOCTYPE html>
-| <html>
-|   <head>
-|   <body>
-|     <p>
-|       <button>
-|         <table>
 resources/webkit01.dat: PASS
 
 resources/webkit02.dat: PASS
index 6a0a08e..7d96db6 100644 (file)
@@ -2,6 +2,18 @@
 
         Reviewed by Adam Barth.
 
+        Fix one more html5lib test case for button scoping
+        https://bugs.webkit.org/show_bug.cgi?id=44583
+
+        I <3 test driven development.
+
+        * html/HTMLTreeBuilder.cpp:
+        (WebCore::HTMLTreeBuilder::processStartTagForInBody):
+
+2010-08-24  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Barth.
+
         Remove HTML5 parser testing infrastructure now that we don't need it
         https://bugs.webkit.org/show_bug.cgi?id=44581
 
index 3904875..24eb62f 100644 (file)
@@ -923,7 +923,7 @@ void HTMLTreeBuilder::processStartTagForInBody(AtomicHTMLToken& token)
         return;
     }
     if (token.name() == tableTag) {
-        if (m_document->parseMode() != Document::Compat && m_tree.openElements()->inScope(pTag))
+        if (m_document->parseMode() != Document::Compat && m_tree.openElements()->inButtonScope(pTag))
             processFakeEndTag(pTag);
         m_tree.insertHTMLElement(token);
         m_framesetOk = false;