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..b58dc9d23e582f6e82a5a66332b9014917f359b3 100644
--- a/java/org/chromium/distiller/webdocument/DomConverter.java
+++ b/java/org/chromium/distiller/webdocument/DomConverter.java
@@ -19,7 +19,6 @@ import org.chromium.distiller.extractors.embeds.ImageExtractor;
 import org.chromium.distiller.extractors.embeds.TwitterExtractor;
 import org.chromium.distiller.extractors.embeds.VimeoExtractor;
 import org.chromium.distiller.extractors.embeds.YouTubeExtractor;
-
 import java.util.ArrayList;

[wychen, 2015/08/01 01:00:20]

nitpick: why deleting the empty line above?

 import java.util.HashSet;
 import java.util.List;
@@ -100,6 +99,11 @@ public class DomConverter implements DomWalker.Visitor {
         }
 
         switch (e.getTagName()) {
+            case "OL":
+            case "UL":
+            case "LI":
+                builder.list(new PlaceHolder(e.getTagName(), PlaceHolder.TagType.START));
+                break;
             case "BR":
                 builder.lineBreak(e);
                 return false;
@@ -142,6 +146,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 PlaceHolder(e.getTagName(), PlaceHolder.TagType.END));
+                    break;
+            }
+        }
         builder.endElement();
     }