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 48 matching lines @@
     ImageQualityController::remove(*this);
     m_imageResource->shutdown();
     LayoutSVGModelObject::willBeDestroyed();
 }
 
 void LayoutSVGImage::updateBoundingBox()
 {
     FloatRect oldBoundaries = m_objectBoundingBox;
 
     SVGLengthContext lengthContext(element());
+    LayoutSize intrinsicSize = m_imageResource->imageSize(style()->effectiveZoom());

[fs, 2016/08/11 15:52:57]

Probably needs to refine this a bit. (The end of)
SVGImagePainter::computeImageViewportSize can probably serve as a reference.

[Shanmuga Pandi, 2016/08/16 10:24:57]

Done.

     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));
+        styleRef().width().isAuto() ? intrinsicSize.width().toFloat() : lengthContext.valueForLength(styleRef().width(), styleRef(), SVGLengthMode::Width),
+        styleRef().height().isAuto() ? intrinsicSize.height().toFloat() : lengthContext.valueForLength(styleRef().height(), styleRef(), SVGLengthMode::Height));
     m_needsBoundariesUpdate |= oldBoundaries != m_objectBoundingBox;
 }
 
 void LayoutSVGImage::layout()
 {
     ASSERT(needsLayout());
     LayoutAnalyzer::Scope analyzer(*this);
 
     // Invalidate all resources of this client if our layout changed.
     if (everHadLayout() && selfNeedsLayout())
@@ skipping 48 matching lines @@
                 updateHitTestResult(result, localLayoutPoint);
                 if (result.addNodeToListBasedTestResult(element(), localLayoutPoint) == StopHitTesting)
                     return true;
             }
         }
     }
 
     return false;
 }
 
 void LayoutSVGImage::imageChanged(WrappedImagePtr, const IntRect*)

[fs, 2016/08/11 16:39:36]

Probably need to make sure to trigger layout when the image changes and
width/height is 'auto'. (More optimally only when dimensions actually end up
changing.)

[Shanmuga Pandi, 2016/08/16 10:24:57]

Done.

 {
     // 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);
 
     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