Chromium Code Reviews| Index: java/org/chromium/distiller/webdocument/WebTable.java |
| diff --git a/java/org/chromium/distiller/webdocument/WebTable.java b/java/org/chromium/distiller/webdocument/WebTable.java |
| index 48ed32d0dc2a5c97a8b33601d1d889228423ab9f..06ee861a4ee7ea44b54156ac807aa2b8190ba16d 100644 |
| --- a/java/org/chromium/distiller/webdocument/WebTable.java |
| +++ b/java/org/chromium/distiller/webdocument/WebTable.java |
| @@ -51,11 +51,13 @@ public class WebTable extends WebElement { |
| cloneAndProcessNode(); |
| } |
| List<String> imgUrls = new ArrayList<>(); |
| - NodeList<Element> imgs = DomUtil.querySelectorAll(cloned, "IMG"); |
| + NodeList<Element> imgs = DomUtil.querySelectorAll(cloned, "IMG, SOURCE"); |
| for (int i = 0; i < imgs.getLength(); i++) { |
| ImageElement ie = (ImageElement) imgs.getItem(i); |
| - imgUrls.add(ie.getSrc()); |
| - imgUrls.addAll(DomUtil.getSrcSetUrls(ie)); |
| + if (!ie.getSrc().isEmpty()) { |
|
mdjones
2017/01/17 17:41:47
nit: same in-line comment as above.
wychen
2017/01/18 17:29:54
Acknowledged.
|
| + imgUrls.add(ie.getSrc()); |
| + } |
| + imgUrls.addAll(DomUtil.getAllSrcSetUrls(ie)); |
| } |
| return imgUrls; |
| } |