Fix for keeping lists structure

java/org/chromium/distiller/webdocument/WebText.java

Index: java/org/chromium/distiller/webdocument/WebText.java
diff --git a/java/org/chromium/distiller/webdocument/WebText.java b/java/org/chromium/distiller/webdocument/WebText.java
index ae81ee6f97f2d9ca41ddfa00be521cf7646ab7bb..86d56140d591e6ccf17c0c809168e4fb55644932 100644
--- a/java/org/chromium/distiller/webdocument/WebText.java
+++ b/java/org/chromium/distiller/webdocument/WebText.java
@@ -69,10 +69,13 @@ public class WebText extends WebElement {
         DomUtil.stripIds(clonedRoot);
         DomUtil.stripFontColorAttributes(clonedRoot);
 
+        Element elementClonedRoot = Element.as(clonedRoot);
         if (textOnly) {
-            return Element.as(clonedRoot).getInnerText();
+            return elementClonedRoot.getInnerText();
+        } else if (elementClonedRoot.getTagName().equals("LI")) {
+            return elementClonedRoot.getInnerHTML();

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

I see why this is needed in this implementation, but would you mind leaving a
comment here saying if an LI element has no child element, the <li></li> tags
would be duplicated?

I think blockquote will also be a special case here.

[dalmirdasilva, 2015/08/01 16:27:04]

Is here the case where we can ask someone else whether we need to use the
innerHTML? A static method inside PlaceHolder which knows all the tags that will
be duplicated. Otherwise we will have to change this code every time we include
a new tag (blockquote) .

         }
-        return Element.as(clonedRoot).getString();
+        return elementClonedRoot.getString();
     }
 
     public List<Node> getTextNodes() {