SVG Image intrinsic size should be used if css style size is 'auto'

third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp

 /*
  * Copyright (C) 2006 Alexander Kellett <lypanov@kde.org>
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007, 2008, 2009 Rob Buis <buis@kde.org>
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) 2010 Patrick Gansterer <paroga@paroga.com>
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 43 matching lines @@
 {
 }
 
 void LayoutSVGImage::willBeDestroyed()
 {
     ImageQualityController::remove(*this);
     m_imageResource->shutdown();
     LayoutSVGModelObject::willBeDestroyed();
 }
 
-void LayoutSVGImage::updateBoundingBox()
+static float resolveWidthForRatio(float height, const FloatSize& intrinsicRatio)
+{
+    return height * intrinsicRatio.width() / intrinsicRatio.height();
+}
+
+static float resolveHeightForRatio(float width, const FloatSize& intrinsicRatio)
+{
+    return width * intrinsicRatio.height() / intrinsicRatio.width();
+}
+
+FloatSize LayoutSVGImage::calculateObjectSize()
+{
+    ImageResource* cachedImage = m_imageResource->cachedImage();
+    if (!cachedImage || cachedImage->errorOccurred())
+        return FloatSize();
+
+    FloatSize intrinsicSize = FloatSize(cachedImage->getImage()->size());
+    if (styleRef().width().isAuto() && styleRef().height().isAuto())
+        return intrinsicSize;
+
+    if (styleRef().height().isAuto())
+        return FloatSize(m_objectBoundingBox.width(), resolveHeightForRatio(m_objectBoundingBox.width(), intrinsicSize));
+
+    if (styleRef().width().isAuto())
+        return FloatSize(resolveWidthForRatio(m_objectBoundingBox.height(), intrinsicSize), m_objectBoundingBox.height());
+
+    return FloatSize();

[fs, 2016/09/14 11:31:52]

This is unreachable? Maybe just assert styleRef().width().isAuto() and return
that size instead?

[Shanmuga Pandi, 2016/09/14 13:15:36]

Done.

+}
+
+bool LayoutSVGImage::updateBoundingBox()
 {
     FloatRect oldBoundaries = m_objectBoundingBox;
 
     SVGLengthContext lengthContext(element());
     m_objectBoundingBox = FloatRect(
         lengthContext.valueForLength(styleRef().svgStyle().x(), styleRef(), SVGLengthMode::Width),
         lengthContext.valueForLength(styleRef().svgStyle().y(), styleRef(), SVGLengthMode::Height),
         lengthContext.valueForLength(styleRef().width(), styleRef(), SVGLengthMode::Width),
         lengthContext.valueForLength(styleRef().height(), styleRef(), SVGLengthMode::Height));
+
+    if (styleRef().width().isAuto() || styleRef().height().isAuto()) {
+        FloatSize intrinsicSize = calculateObjectSize();
+        if (styleRef().width().isAuto())

[fs, 2016/09/14 11:31:52]

This branching looks redundant - couldn't this just do:

m_objectBoundingBox.setSize(calculateObjectSize());

? (Might need to return m_objectBoundingBox.size() in the error-case in
calculateObjectSize.)

[Shanmuga Pandi, 2016/09/14 13:15:36]

Done.

+            m_objectBoundingBox.setWidth(intrinsicSize.width());
+        if (styleRef().height().isAuto())
+            m_objectBoundingBox.setHeight(intrinsicSize.height());
+    }
     m_needsBoundariesUpdate |= oldBoundaries != m_objectBoundingBox;
     if (element())
         element()->setNeedsResizeObserverUpdate();
+
+    return oldBoundaries.size() != m_objectBoundingBox.size();
 }
 
 void LayoutSVGImage::layout()
 {
     ASSERT(needsLayout());
     LayoutAnalyzer::Scope analyzer(*this);
 
     // Invalidate all resources of this client if our layout changed.
     if (everHadLayout() && selfNeedsLayout())
         SVGResourcesCache::clientLayoutChanged(this);
@@ skipping 54 matching lines @@
     return false;
 }
 
 void LayoutSVGImage::imageChanged(WrappedImagePtr, const IntRect*)
 {
     // Notify parent resources that we've changed. This also invalidates
     // references from resources (filters) that may have a cached
     // representation of this image/layout object.
     LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(this, false);
 
+    if (styleRef().width().isAuto() || styleRef().height().isAuto()) {
+        if (updateBoundingBox())
+            setNeedsLayout(LayoutInvalidationReason::SizeChanged);
+    }
+
     setShouldDoFullPaintInvalidation();
 }
 
 void LayoutSVGImage::addOutlineRects(Vector<LayoutRect>& rects, const LayoutPoint&, IncludeBlockVisualOverflowOrNot) const
 {
     // this is called from paint() after the localTransform has already been applied
     rects.append(LayoutRect(paintInvalidationRectInLocalSVGCoordinates()));
 }
 
 } // namespace blink