Bug 259905 – [SVG] in img element doesn't render correctly when height is changed dynamically

• Component: SVG
• Status: NEW

[Ahmad Saleem]

Hi Team,

Came across failing test case from StackOverflow (link don't work) while going through Blink's commit rabbit hole:

TestCase: https://jsfiddle.net/p9d3jmk2/

^ Modified with placeholder image 200*200. Not stretching properly in Safari.

Blink Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=170437

Local patch does not fix this on WebKit ToT.

Thanks!

[Radar WebKit Bug Importer]

<rdar://problem/113880536>

[Karl Dubost]

Ahmad, I seem to have the same behavior on all three browsers

[Ahmad Saleem]

Created attachment 467551 [details]
"Tall@ - Chrome Canary vs WebKit ToT

@Karl - I get this where 'Tall' does not do anything.

[Karl Dubost]

re-tested and you are right. 

https://searchfox.org/wubkat/rev/df7a74a8f788e5e1ea909d9e843a674d0e1e50a8/Source/WebCore/svg/graphics/SVGImage.cpp#322-327

[Karl Dubost]

fwiw the chromium patch applied to WebKit doesn't fix it.

[Ahmad Saleem]

void setContainerSize(const IntSize& containerSize) {
        if (m_containerSize != containerSize)
            setNeedsLayout();
        m_containerSize = containerSize;
    }

Based on Chromium patch but it does not work.