[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/4.1/kdelibs/khtml/html
From:       Germain Garand <germain () ebooksfrance ! org>
Date:       2008-09-15 4:27:50
Message-ID: 1221452870.810467.19238.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 861087 by ggarand:

automatically merged revision 861054:
.doing strict dtd checking for inlines in strict mode turns out to be
 too optimistic. It breaks sites.

.no popBlock for dl/dt errors. It doesn't match what other browsers do.

BUG: 170650

 M  +2 -5      dtd.cpp  
 M  +0 -8      htmlparser.cpp  


--- branches/KDE/4.1/kdelibs/khtml/html/dtd.cpp #861086:861087
@@ -474,14 +474,11 @@
     case ID_FONT:
     case ID_LEGEND:
     case ID_Q:
+    case ID_A:
     case ID_NOBR:
     case ID_WBR:
         // %inline *
-        return check_inline(childID, strict) ||
-               (!strict && check_block(childID, strict));
-    case ID_A:
-        // A: %inline * (but even strict sites expect %flow)
-        return check_flow(childID, strict);
+        return check_inline(childID, strict) || check_block(childID, strict);
     case ID_P:
         // P: %inline *
         return check_inline(childID, strict) ||
--- branches/KDE/4.1/kdelibs/khtml/html/htmlparser.cpp #861086:861087
@@ -801,14 +801,6 @@
             insertNode(e);
             handled = true;
             break;
-        case ID_DL:
-            popBlock(ID_DL);
-            handled = true;
-            break;
-        case ID_DT:
-            popBlock(ID_DT);
-            handled = true;
-            break;
         case ID_FORM:
             popBlock(ID_FORM);
             handled = true;
[prev in list] [next in list] [prev in thread] [next in thread]