Retain image sizes

java/org/chromium/distiller/extractors/embeds/ImageExtractor.java

Index: java/org/chromium/distiller/extractors/embeds/ImageExtractor.java
diff --git a/java/org/chromium/distiller/extractors/embeds/ImageExtractor.java b/java/org/chromium/distiller/extractors/embeds/ImageExtractor.java
index a450ea9d23f276a71b664a0e78254dee7159fdeb..14801271e155377f2f1324738e9e5c50d3e0c1b2 100644
--- a/java/org/chromium/distiller/extractors/embeds/ImageExtractor.java
+++ b/java/org/chromium/distiller/extractors/embeds/ImageExtractor.java
@@ -33,11 +33,21 @@ public class ImageExtractor implements EmbedExtractor {
             return null;
         }
         String imgSrc = "";
+        // Getting OffSetWidth/Height as default values, even they are
+        // affected by padding, border, etc.
+        int width = e.getOffsetWidth();
+        int height = e.getOffsetHeight();
         if ("IMG".equals(e.getTagName())) {
-            // This will get the absolute URL of the image.
-            imgSrc = ImageElement.as(e).getSrc();
+            // This will get the absolute URL of the image and
+            // the real image dimension.

[wychen, 2016/03/18 08:20:25]

"Real" is kind of ambiguous here. Maybe "displayed"?

[dalmirsilva, 2016/03/18 17:20:37]

Done.

+            ImageElement imageElement = ImageElement.as(e);
+            imgSrc = imageElement.getSrc();
+            // As an ImageElement is manipulated here, it is possible
+            // to get the real dimensions.
+            width = imageElement.getWidth();

[wychen, 2016/03/18 08:20:25]

Do we always know the size at this point? Late loaded images might not have
dimensions yet. Some sites only load images when it's close to the viewport. We
might want to handle 0 somewhere, just in case.

[dalmirsilva, 2016/03/18 17:20:37]

Done.

+            height = imageElement.getHeight();
         }
 
-        return new WebImage(e, e.getOffsetWidth(), e.getOffsetHeight(), imgSrc);
+        return new WebImage(e, width, height, imgSrc);
     }
 }