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

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

 /*
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 76 matching lines @@
     }
 
     FloatRect oldViewport = m_viewport;
 
     // Cache viewport boundaries
     SVGLengthContext lengthContext(foreign);
     FloatPoint viewportLocation(
         lengthContext.valueForLength(styleRef().svgStyle().x(), styleRef(), SVGLengthMode::Width),
         lengthContext.valueForLength(styleRef().svgStyle().y(), styleRef(), SVGLengthMode::Height));
     m_viewport = FloatRect(viewportLocation, FloatSize(
-        lengthContext.valueForLength(styleRef().width(), styleRef(), SVGLengthMode::Width),
-        lengthContext.valueForLength(styleRef().height(), styleRef(), SVGLengthMode::Height)));
+        styleRef().width().isAuto() ? foreign->width()->currentValue()->value(lengthContext) : lengthContext.valueForLength(styleRef().width(), styleRef(), SVGLengthMode::Width),

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

I guess 'auto' here would mean some form of shrink-wrap or adapting to whatever
is within the fO. I'd suggest to leave it alone for now though.

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

Acknowledged.

+        styleRef().height().isAuto() ? foreign->height()->currentValue()->value(lengthContext) : lengthContext.valueForLength(styleRef().height(), styleRef(), SVGLengthMode::Height)));
     if (!updateCachedBoundariesInParents)
         updateCachedBoundariesInParents = oldViewport != m_viewport;
 
     // Set box origin to the foreignObject x/y translation, so positioned objects in XHTML content get correct
     // positions. A regular LayoutBoxModelObject would pull this information from ComputedStyle - in SVG those
     // properties are ignored for non <svg> elements, so we mimic what happens when specifying them through CSS.
 
     // FIXME: Investigate in location rounding issues - only affects LayoutSVGForeignObject & LayoutSVGText
     setLocation(roundedIntPoint(viewportLocation));
 
@@ skipping 27 matching lines @@
         return false;
 
     // FOs establish a stacking context, so we need to hit-test all layers.
     HitTestLocation hitTestLocation(localPoint);
     return LayoutBlock::nodeAtPoint(result, hitTestLocation, LayoutPoint(), HitTestForeground)
         || LayoutBlock::nodeAtPoint(result, hitTestLocation, LayoutPoint(), HitTestFloat)
         || LayoutBlock::nodeAtPoint(result, hitTestLocation, LayoutPoint(), HitTestChildBlockBackgrounds);
 }
 
 } // namespace blink