Cache the viewport size inside SVGLengthContext

[Simon Fraser (smfr)]

Cache the viewport size inside SVGLengthContext

[Simon Fraser (smfr)]

Created attachment 458933 [details]
Patch

[Nikolas Zimmermann]

Ok, with me. In LBSE downstream we directly cache the whole SVGLengthContext in SVGElement, for the same reason... (some FIXMES already present in the code regarding that)

[Simon Fraser (smfr)]

Created attachment 458972 [details]
Patch

[zalan]

Comment on attachment 458972 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=458972&action=review

> Source/WebCore/svg/SVGLengthContext.cpp:290
> -        return false;
> +        return std::nullopt;

return { };

> Source/WebCore/svg/SVGLengthContext.cpp:314
> +        return std::nullopt;

here too

[EWS]

Committed r293989 (250426@main): <https://commits.webkit.org/250426@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 458972 [details].

[Radar WebKit Bug Importer]

<rdar://problem/92984418>