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

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

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp
index 91867b267e3846e7871142f9b3e0af1f11131e9a..e1aa94a1a815e441a27454b33e8a70237c8aa042 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp
@@ -66,11 +66,12 @@ void LayoutSVGImage::updateBoundingBox()
     FloatRect oldBoundaries = m_objectBoundingBox;
 
     SVGLengthContext lengthContext(element());
+    SVGImageElement* image = toSVGImageElement(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));
+        styleRef().width().isAuto() ? image->width()->currentValue()->value(lengthContext) : lengthContext.valueForLength(styleRef().width(), styleRef(), SVGLengthMode::Width),

[fs, 2016/08/10 18:01:34]

No, this is not what should happen. To quote the relevant suggestion from the
bug: "...I think 'auto' should mean use the intrinsic width of the referenced
image." Notice that last part. So if we have an <image> referencing a bitmap
with dimensions 1024x768, then 'width: auto' would mean a used value of '1024'
(and similarly '768' for height.) I'm not sure if anything in particular should
be done for when SVG images are referenced - maybe, maybe not.

[Shanmuga Pandi, 2016/08/11 05:54:20]

Thank you fs@ for explaining in detail.
Done the changes.

+        styleRef().height().isAuto() ? image->height()->currentValue()->value(lengthContext) : lengthContext.valueForLength(styleRef().height(), styleRef(), SVGLengthMode::Height));
     m_needsBoundariesUpdate |= oldBoundaries != m_objectBoundingBox;
 }