Fix for keeping lists structure

java/org/chromium/distiller/webdocument/DomConverter.java

Index: java/org/chromium/distiller/webdocument/DomConverter.java
diff --git a/java/org/chromium/distiller/webdocument/DomConverter.java b/java/org/chromium/distiller/webdocument/DomConverter.java
index 96c9c4b21a1cd3e240b4a2468ffc34e81f519bde..7dc7244afb8ba11e2e8c2588ec48610d1903178b 100644
--- a/java/org/chromium/distiller/webdocument/DomConverter.java
+++ b/java/org/chromium/distiller/webdocument/DomConverter.java
@@ -100,6 +100,11 @@ public class DomConverter implements DomWalker.Visitor {
         }
 
         switch (e.getTagName()) {
+            case "OL":
+            case "UL":
+            case "LI":
+                builder.tag(new WebTag(e.getTagName(), WebTag.TagType.START));

[wychen, 2015/08/11 00:12:15]

We could move this out of the switch and use canBeNested() instead.

[Marcelo Correa, 2015/08/11 00:21:01]

Sure, But this wouldn't change the pattern how the elements are filtered?
I mean, excluding the embed extractor and hidden elements, everything else is
filtered by the switch.

+                break;
             case "BR":
                 builder.lineBreak(e);
                 return false;
@@ -142,6 +147,16 @@ public class DomConverter implements DomWalker.Visitor {
 
     @Override
     public void exit(Node n) {
+        if (n.getNodeType() == Node.ELEMENT_NODE) {
+            Element e = Element.as(n);
+            switch (e.getTagName()) {
+                case "OL":
+                case "UL":
+                case "LI":
+                    builder.tag(new WebTag(e.getTagName(), WebTag.TagType.END));

[wychen, 2015/08/11 00:12:15]

And here.

+                    break;
+            }
+        }
         builder.endElement();
     }