Add support for figure element

javatests/org/chromium/distiller/EmbedExtractorTest.java

 // Copyright 2014 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;
 
 import com.google.gwt.dom.client.Style;
 import org.chromium.distiller.webdocument.WebEmbed;
 import org.chromium.distiller.webdocument.WebImage;
 import org.chromium.distiller.extractors.embeds.EmbedExtractor;
@@ skipping 362 matching lines @@
         ImageElement image = TestUtil.createImage();
         image.setSrc(IMAGE_BASE64);
         image.getStyle().setHeight(1, Style.Unit.CM);
         mBody.appendChild(image);
         EmbedExtractor extractor = new ImageExtractor();
         WebImage result = (WebImage) extractor.extract(image);
         assertNotNull(result);
         assertEquals(38, result.getHeight());
         assertEquals(38, result.getWidth());
     }
+
+    public void testFigureWithoutCaption() {
+        ImageElement image = TestUtil.createImage();
+        image.setSrc("http://wwww.someimage.com/image.jpeg");

[wychen, 2016/05/31 21:28:04]

We should use example.com for the domain.

[marcelorcorrea, 2016/05/31 21:46:44]

Done.

+        image.setAttribute("width", "100");
+        image.setAttribute("height", "100");
+        Element figure = Document.get().createElement("FIGURE");
+        figure.appendChild(image);
+        mBody.appendChild(figure);
+
+        EmbedExtractor extractor = new ImageExtractor();
+        WebImage result = (WebImage) extractor.extract(figure);
+        String got = result.generateOutput(false);
+        String expected =
+                "<figure>" +
+                    "<img src=\"http://wwww.someimage.com/image.jpeg\"" +
+                        " width=\"100\" height=\"100\">" +
+                "</figure>";
+        assertNotNull(result);
+        assertEquals(100, result.getHeight());
+        assertEquals(100, result.getWidth());
+        assertEquals(expected, got);
+    }
+
+    public void testFigureWithCaption() {

[wychen, 2016/05/31 21:28:04]

Add one more test without <img>?

[marcelorcorrea, 2016/05/31 21:46:44]

Done.

+        ImageElement image = TestUtil.createImage();
+        image.setSrc("http://wwww.someimage.com/image.jpeg");
+        image.setAttribute("width", "100");
+        image.setAttribute("height", "100");
+        Element figure = Document.get().createElement("FIGURE");
+        figure.appendChild(image);
+        Element figcaption = Document.get().createElement("FIGCAPTION");
+        figcaption.setInnerHTML("This is a caption");
+        figure.appendChild(figcaption);
+        mBody.appendChild(figure);
+
+        EmbedExtractor extractor = new ImageExtractor();
+        WebImage result = (WebImage) extractor.extract(figure);
+        String got = result.generateOutput(false);
+        String expected =
+                "<figure>" +
+                    "<img src=\"http://wwww.someimage.com/image.jpeg\"" +
+                        " width=\"100\" height=\"100\">" +
+                    "<figcaption>This is a caption</figcaption>" +
+                "</figure>";
+        assertNotNull(result);
+        assertEquals(100, result.getHeight());
+        assertEquals(100, result.getWidth());
+        assertEquals(expected, got);
+    }
 }