Bug 259905 - [SVG] in img element doesn't render correctly when height is changed dynamically
Summary: [SVG] in img element doesn't render correctly when height is changed dynamically
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2023-08-07 17:16 PDT by Ahmad Saleem
Modified: 2023-09-10 03:40 PDT (History)
4 users (show)

See Also:

Attachments
"Tall@ - Chrome Canary vs WebKit ToT (143.01 KB, image/png)
2023-09-05 05:10 PDT, Ahmad Saleem 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-08-07 17:16:03 PDT 
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!
Comment 1 Radar WebKit Bug Importer 2023-08-14 17:17:12 PDT 
<rdar://problem/113880536>
Comment 2 Karl Dubost 2023-08-20 18:50:42 PDT 
Ahmad, I seem to have the same behavior on all three browsers
Comment 3 Ahmad Saleem 2023-09-05 05:10:36 PDT 
Created attachment 467551 [details]
"Tall@ - Chrome Canary vs WebKit ToT

@Karl - I get this where 'Tall' does not do anything.
Comment 5 Karl Dubost 2023-09-05 22:34:42 PDT 
fwiw the chromium patch applied to WebKit doesn't fix it.
Comment 6 Ahmad Saleem 2023-09-10 03:40:45 PDT 
void setContainerSize(const IntSize& containerSize) {
        if (m_containerSize != containerSize)
            setNeedsLayout();
        m_containerSize = containerSize;
    }

Based on Chromium patch but it does not work.