[Webkit-unassigned] [Bug 130127] Nested use of same SVG resource fails

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=130127

Ahmad Saleem <ahmad.saleem792 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rniwa at webkit.org

--- Comment #3 from Ahmad Saleem <ahmad.saleem792 at gmail.com> ---
(In reply to Ahmad Saleem from comment #1)
> It is still happening with Safari 16.2 and Safari Technology Preview 160, it
> was fixed in Blink with following commit:
> 
> Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=175129
> & https://src.chromium.org/viewvc/blink?revision=174862&view=revision
> 
> WebKit Source (Example) -
> https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/
> SVGResourcesCycleSolver.cpp#53
> 
> Just wanted to share this. Might be bit out of my area of expertise to try
> to merge this and don't want to cause collision with on-going LBSE work as
> well (if it reaches in this realm). Just wanted to share. Thanks!

We only need to do: https://src.chromium.org/viewvc/blink?view=revision&revision=175129

^ I tried but I think our code has also evolved, so it is not 1-1, there are still some similarities but we have WeakHashSet intro in this code as well.

@Ryosuke did some WeakHashSet intro in various places of code. Might be something you can have a look?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230514/9e380697/attachment.htm>