262192 2023-09-27 09:07:41 -0700 A GitHub project page with embedded SVG takes an extremely long time to load 2023-11-03 01:45:28 -0700 1 1 1 Unclassified WebKit SVG Safari 17 All All RESOLVED CONFIGURATION CHANGED https://bugs.webkit.org/show_bug.cgi?id=262343 https://bugs.webkit.org/show_bug.cgi?id=264076 BrowserCompat, InRadar P2 Normal --- 1 cueball045 webkit-unassigned ahmad.saleem792 ap karlcow mmaxfield rniwa sabouhallawa simon.fraser webkit-bug-importer zimmermann oldest_to_newest 1980531 0 cueball045 2023-09-27 09:07:41 -0700 Website: https://github.com/facebookincubator/below The website takes a long time to load, the page hangs. Navigating to sub-links is also slow. This is a WebKit specific behavior, the website loads fine on Chrome. I also tried disabling all the extensions but this looks like a bug with WebKit. 1980744 1 ap 2023-09-27 17:56:52 -0700 Thsnk you for the report! As this is reported against Safari 17, do you happen to know if this was happening with Safari 16.x before? It's taking time under LegacyRenderSVGRoot::layout -> SVGTextMetricsBuilder::walkTree -> WebCore::Font::applyTransforms. 1980758 2 cueball045 2023-09-27 18:15:41 -0700 Thanks for the prompt response. Yes, this was happening in Safari 16.x as well. I had reported this for Safari 16 using Feedback Assistant but I should have known better. 1981016 3 simon.fraser 2023-09-28 09:49:09 -0700 Lots of time under: 2404 WebCore::RenderBlockFlow::layoutBlockChildren(bool, WebCore::LayoutUnit&) (in WebCore) + 4960 [0x61767b2b0] 2404 WebCore::LegacyRenderSVGRoot::layout() (in WebCore) + 1094 [0x6178bfe86] 2404 WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) (in WebCore) + 706 [0x6178a03d2] 2404 WebCore::LegacyRenderSVGContainer::layout() (in WebCore) + 269 [0x6178b5a6d] 2404 WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) (in WebCore) + 706 [0x6178a03d2] 2402 WebCore::RenderSVGText::layout() (in WebCore) + 1309 [0x61558439d] ! 2398 WebCore::SVGTextMetricsBuilder::walkTree(WebCore::RenderElement&, WebCore::RenderSVGInlineText*, WebCore::MeasureTextData*) (in WebCore) + 127 [0x6178b3b4f] ! : 2329 WebCore::SVGTextMetricsBuilder::walkTree(WebCore::RenderElement&, WebCore::RenderSVGInlineText*, WebCore::MeasureTextData*) (in WebCore) + 127 [0x6178b3b4f] ! : | 2279 WebCore::SVGTextMetricsBuilder::walkTree(WebCore::RenderElement&, WebCore::RenderSVGInlineText*, WebCore::MeasureTextData*) (in WebCore) + 92 [0x6178b3b2c] ! : | + 1997 WebCore::SVGTextMetricsBuilder::measureTextRenderer(WebCore::RenderSVGInlineText&, WebCore::MeasureTextData*) (in WebCore) + 1358 [0x6178b31ce] ! : | + ! 1404 WebCore::WidthIterator::advance(unsigned int, WebCore::GlyphBuffer&) (in WebCore) + 188 [0x6174737cc] ! : | + ! : 1098 WebCore::WidthIterator::advanceInternal<WebCore::Latin1TextIterator>(WebCore::Latin1TextIterator&, WebCore::GlyphBuffer&) (in WebCore) + 4011 [0x617474c7b] ! : | + ! : | 1054 WebCore::WidthIterator::commitCurrentFontRange(WebCore::AdvanceInternalState&) (in WebCore) + 963 [0x6174722d3] 1981017 4 webkit-bug-importer 2023-09-28 09:49:20 -0700 <rdar://problem/116188712> 1981023 5 ahmad.saleem792 2023-09-28 10:27:49 -0700 One potential merge: https://src.chromium.org/viewvc/blink?view=revision&revision=168005 > Eliminate recursion in SVGTextMetricsBuilder::walkTree 1989524 6 ahmad.saleem792 2023-11-02 14:42:37 -0700 I think it is fixed on WebKit ToT (270133@main). 1989527 7 cueball045 2023-11-02 14:50:37 -0700 Is there a way to verify this, maybe in Safari TP? I am not sure which commit fixed it, so cannot check in release notes. 1989529 8 ahmad.saleem792 2023-11-02 14:52:02 -0700 (In reply to cueball045 from comment #7) > Is there a way to verify this, maybe in Safari TP? I am not sure which > commit fixed it, so cannot check in release notes. I think it might be fixed because of following commit: https://commits.webkit.org/270110@main Which is not in Safari Technology Preview. It might be in STP183 or STP184 or even STP185. 1989530 9 cueball045 2023-11-02 14:53:09 -0700 That's super helpful. Thanks a lot! 1989634 10 rniwa 2023-11-03 01:45:28 -0700 Yeah, this appears to be fixed on trunk.