Add support for figure element

java/org/chromium/distiller/webdocument/WebFigure.java

Index: java/org/chromium/distiller/webdocument/WebFigure.java
diff --git a/java/org/chromium/distiller/webdocument/WebFigure.java b/java/org/chromium/distiller/webdocument/WebFigure.java
new file mode 100644
index 0000000000000000000000000000000000000000..c7ae6ae2d8310f8b8fa064c67072110463d12c7c
--- /dev/null
+++ b/java/org/chromium/distiller/webdocument/WebFigure.java
@@ -0,0 +1,26 @@
+package org.chromium.distiller.webdocument;
+
+import com.google.gwt.dom.client.Document;
+import com.google.gwt.dom.client.Element;
+
+public class WebFigure extends WebImage {
+
+    private String figCaption;
+
+    public WebFigure(Element e, int w, int h, String src, String caption) {
+        super(e, w, h, src);
+        this.figCaption = caption;
+    }
+
+    @Override
+    public String generateOutput(boolean textOnly) {
+        Element fig = Document.get().createElement("FIGURE");
+        fig.setInnerHTML(super.generateOutput(textOnly));
+        if (!figCaption.isEmpty()) {
+            Element cap = Document.get().createElement("FIGCAPTION");
+            cap.setInnerHTML(figCaption);

[wychen, 2016/05/31 22:14:46]

The caption is obtained from innerText, but set as innerHTML? If we want to keep
text only, probably wrap inside <p> here. If we want to keep rich structure,
sanitize it here.

[marcelorcorrea, 2016/06/01 18:09:54]

Right. We've changed to set text only since according to this:
https://www.w3.org/TR/html5/grouping-content.html#the-figure-element
wouldn't make sense to preserve HTML.
Usually the figure element is used like this:
<figure>
    <div>
        <img src="http://img source">
        <figcaption>example caption</figcaption>
    </div>
</figure>

+            fig.appendChild(cap);
+        }
+        return fig.getString();
+    }
+}