Bug 256702 – Resolve percentage in <use> against the instance's viewport element

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Ahmad Saleem]

Hi Team,

While going through Blink's commit, I came across another potential merge to fix another failing test (in the URL field).

Blink Commit - https://chromium.googlesource.com/chromium/src/+/b34ff6fe2d5842a63e2c66f68f050f0cf9163010

WebKit Source - https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/LegacyRenderSVGTransformableContainer.cpp#61

I don't know how to fix LBSE so I think till we don't have LBSE enabled, we can still fix it in Legacy.

Just wanted to raise so we can fix it. (It might not be single merge, because we have 'bool' function while it is different in Blink).

Thanks!

[Ahmad Saleem]

(In reply to Ahmad Saleem from comment #0)
> Hi Team,
> 
> While going through Blink's commit, I came across another potential merge to
> fix another failing test (in the URL field).
> 
> Blink Commit -
> https://chromium.googlesource.com/chromium/src/+/
> b34ff6fe2d5842a63e2c66f68f050f0cf9163010
> 
> WebKit Source -
> https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/
> LegacyRenderSVGTransformableContainer.cpp#61
> 
> I don't know how to fix LBSE so I think till we don't have LBSE enabled, we
> can still fix it in Legacy.
> 
> Just wanted to raise so we can fix it. (It might not be single merge,
> because we have 'bool' function while it is different in Blink).
> 
> Thanks!

Doing this leads to following error:

'no matching constructor for initialization of 'WebCore::SVGLengthContext''.

[Radar WebKit Bug Importer]

<rdar://problem/109565397>

[EWS]

Committed 264596@main (590f8fce92df): <https://commits.webkit.org/264596@main>

Reviewed commits have been landed. Closing PR #14160 and removing active labels.

[Said Abou-Hallawa]

*** Bug 128589 has been marked as a duplicate of this bug. ***