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..481c4d36c057f2f7c325988fdbd695108eb25788 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.list(new WebTag(e.getTagName(), WebTag.TagType.START));

[wychen, 2015/08/04 02:37:01]

If we skip this when there's no child element, then we don't need to special
case LI in generateOutput().

[Marcelo Correa, 2015/08/04 16:34:48]

I don't think we can do such thing.
There are a few cases when this solution is not going to handle. Let say
we have this HTML:
<ol>
  <li>
    Text
    <p>Text</p>
  </li>
<ol>
If we decide to not include de LI tag, it will end up with a HTML like
this:
<ol>
  <li>Text</li>
  <p>Text</p>
</ol>
If we decide to include the LI, the final HTML will look like this:
<ol>
  <li>
    <li>Text</li>
    <p>Text</p>
  </li>
</ol>
Both cases we have to avoid the extra LI in the WebText class. 
We don't think it is possible to cover all cases only with logic here.
 

[wychen, 2015/08/05 00:01:02]

I see. You are right. The tag is copied to WebText when the content is a text
node.

+                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.list(new WebTag(e.getTagName(), WebTag.TagType.END));
+                    break;
+            }
+        }
         builder.endElement();
     }