Fix for keeping lists structure

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.distiller.webdocument;
 
 import com.google.gwt.dom.client.Element;
 import org.chromium.distiller.DomUtil;
 import org.chromium.distiller.TreeCloneBuilder;
 import org.chromium.distiller.labels.DefaultLabels;
@@ skipping 51 matching lines @@
             Node parentClone = getTextNodes().get(0).getParentElement().cloneNode(false);
             parentClone.appendChild(clonedRoot);
             clonedRoot = parentClone;
         }
 
         // Make sure links are absolute and IDs are gone.
         DomUtil.makeAllLinksAbsolute(clonedRoot);
         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() {
         return allTextNodes.subList(start, end);
     }
 
     public void addLabel(String s) {
         labels.add(s);
     }
 
@@ skipping 40 matching lines @@
     }
 
     public void setGroupNumber(int group) {
         groupNumber = group;
     }
 
     public int getGroupNumber() {
         return groupNumber;
     }
 }