263351 2023-10-18 17:43:21 -0700 [SVG] Enable approximate repaint bounding box computation 2024-11-12 12:47:35 -0800 1 1 1 Unclassified WebKit SVG WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=277624 https://bugs.webkit.org/show_bug.cgi?id=278551 https://bugs.webkit.org/show_bug.cgi?id=279105 InRadar P2 Normal --- 1 ysuzuki ysuzuki fujii sabouhallawa webkit-bug-importer zimmermann oldest_to_newest 1986026 0 ysuzuki 2023-10-18 17:43:21 -0700 ... 1986027 1 webkit-bug-importer 2023-10-18 17:43:44 -0700 <rdar://problem/117175423> 1986029 2 ysuzuki 2023-10-18 17:46:10 -0700 Pull request: https://github.com/WebKit/WebKit/pull/19262 1986071 3 ews-feeder 2023-10-18 21:30:02 -0700 Committed 269500@main (2f7437fb0267): <https://commits.webkit.org/269500@main> Reviewed commits have been landed. Closing PR #19262 and removing active labels. 1986215 4 fujii 2023-10-19 12:46:07 -0700 http/tests/svg/use-custom-property-inheritance.html is crashing after this change. https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fsvg%2Fuse-custom-property-inheritance.html&style=debug https://ews-build.s3-us-west-2.amazonaws.com/macOS-AppleSilicon-Ventura-Debug-WK2-Tests-EWS/ca059f9b-18153/http/tests/svg/use-custom-property-inheritance-stderr.txt ASSERTION FAILED: !renderer.hasNonScalingStroke() /Volumes/Data/worker/macOS-AppleSilicon-Ventura-Debug-Build-EWS/build/Source/WebCore/rendering/svg/SVGRenderSupport.cpp(577) : auto WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(const WebCore::RenderElement &)::(anonymous class)::operator()(const auto &, WebCore::FloatRect) const [renderer:auto = WebCore::LegacyRenderSVGShape] 1 0x12e6cf28c WTFCrash 2 0x282ada7e0 WebCore::BaseAudioContext::isGraphOwner() const 3 0x2856211f8 WebCore::FloatRect WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore::RenderElement const&)::$_2::operator()<WebCore::LegacyRenderSVGShape>(WebCore::LegacyRenderSVGShape const&, WebCore::FloatRect) const 4 0x2856210a0 WebCore::FloatRect WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore::RenderElement const&)::$_1::operator()<WebCore::LegacyRenderSVGShape>(WebCore::LegacyRenderSVGShape const&, WebCore::FloatRect) const 5 0x2856137dc auto WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore::RenderElement const&)::$_0::operator()<WebCore::LegacyRenderSVGShape>(WebCore::LegacyRenderSVGShape const&) const 6 0x285613608 WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore::RenderElement const&) 7 0x28564b904 WebCore::LegacyRenderSVGShape::calculateApproximateStrokeBoundingBox() const 8 0x28564a644 WebCore::LegacyRenderSVGShape::updateRepaintBoundingBox() 9 0x28564a540 WebCore::LegacyRenderSVGShape::layout() 10 0x285611acc WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) 11 0x28563c9ac WebCore::LegacyRenderSVGContainer::layout() 12 0x285611acc WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) 13 0x28563c9ac WebCore::LegacyRenderSVGContainer::layout() 14 0x285611acc WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) 15 0x28564839c WebCore::LegacyRenderSVGRoot::layout() 16 0x2847d3384 WebCore::LocalFrameViewLayoutContext::performLayout() 17 0x2847b160c WebCore::LocalFrameViewLayoutContext::layout() 18 0x2847cca74 WebCore::LocalFrameView::updateLayoutAndStyleIfNeededRecursive() 19 0x284832188 WebCore::Page::layoutIfNeeded() 20 0x284832ca4 WebCore::Page::updateRendering() 21 0x11a3a5aac WebKit::WebPage::updateRendering() 22 0x119443dc8 WebKit::TiledCoreAnimationDrawingArea::updateRendering(WebKit::TiledCoreAnimationDrawingArea::UpdateRenderingType) 23 0x119443cc8 WebKit::TiledCoreAnimationDrawingArea::forceRepaint() 24 0x11a3a462c WebKit::WebPage::forceRepaintWithoutCallback() 25 0x119cbec04 WKBundlePageForceRepaint 26 0x10184959c WTR::InjectedBundlePage::dump() 27 0x10184d988 WTR::dumpAfterWaitAttributeIsRemoved(OpaqueWKBundlePage const*) 28 0x101848de4 WTR::InjectedBundlePage::frameDidChangeLocation(OpaqueWKBundleFrame const*) 29 0x101847250 WTR::InjectedBundlePage::didFinishLoadForFrame(OpaqueWKBundleFrame const*) 30 0x101845e1c WTR::InjectedBundlePage::didFinishLoadForFrame(OpaqueWKBundlePage const*, OpaqueWKBundleFrame const*, void const**, void const*) 31 0x119cafa18 WebKit::InjectedBundlePageLoaderClient::didFinishLoadForFrame(WebKit::WebPage&, WebKit::WebFrame&, WTF::RefPtr<API::Object, WTF::RawPtrTraits<API::Object>, WTF::DefaultRefDerefTraits<API::Object>>&) com.apple.WebKit.WebContent.Development terminated (pid 5864) for reason: crash LEAK: 1 WebPageProxy 1986245 5 ysuzuki 2023-10-19 13:41:53 -0700 (In reply to Fujii Hironori from comment #4) > http/tests/svg/use-custom-property-inheritance.html is crashing after this > change. > https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fsvg%2Fuse- > custom-property-inheritance.html&style=debug > > https://ews-build.s3-us-west-2.amazonaws.com/macOS-AppleSilicon-Ventura- > Debug-WK2-Tests-EWS/ca059f9b-18153/http/tests/svg/use-custom-property- > inheritance-stderr.txt > > ASSERTION FAILED: !renderer.hasNonScalingStroke() > /Volumes/Data/worker/macOS-AppleSilicon-Ventura-Debug-Build-EWS/build/Source/ > WebCore/rendering/svg/SVGRenderSupport.cpp(577) : auto > WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(const > WebCore::RenderElement &)::(anonymous class)::operator()(const auto &, > WebCore::FloatRect) const [renderer:auto = WebCore::LegacyRenderSVGShape] > 1 0x12e6cf28c WTFCrash > 2 0x282ada7e0 WebCore::BaseAudioContext::isGraphOwner() const > 3 0x2856211f8 WebCore::FloatRect > WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore:: > RenderElement > const&)::$_2::operator()<WebCore::LegacyRenderSVGShape>(WebCore:: > LegacyRenderSVGShape const&, WebCore::FloatRect) const > 4 0x2856210a0 WebCore::FloatRect > WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore:: > RenderElement > const&)::$_1::operator()<WebCore::LegacyRenderSVGShape>(WebCore:: > LegacyRenderSVGShape const&, WebCore::FloatRect) const > 5 0x2856137dc auto > WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore:: > RenderElement > const&)::$_0::operator()<WebCore::LegacyRenderSVGShape>(WebCore:: > LegacyRenderSVGShape const&) const > 6 0x285613608 > WebCore::SVGRenderSupport::calculateApproximateStrokeBoundingBox(WebCore:: > RenderElement const&) > 7 0x28564b904 > WebCore::LegacyRenderSVGShape::calculateApproximateStrokeBoundingBox() const > 8 0x28564a644 WebCore::LegacyRenderSVGShape::updateRepaintBoundingBox() > 9 0x28564a540 WebCore::LegacyRenderSVGShape::layout() > 10 0x285611acc > WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) > 11 0x28563c9ac WebCore::LegacyRenderSVGContainer::layout() > 12 0x285611acc > WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) > 13 0x28563c9ac WebCore::LegacyRenderSVGContainer::layout() > 14 0x285611acc > WebCore::SVGRenderSupport::layoutChildren(WebCore::RenderElement&, bool) > 15 0x28564839c WebCore::LegacyRenderSVGRoot::layout() > 16 0x2847d3384 WebCore::LocalFrameViewLayoutContext::performLayout() > 17 0x2847b160c WebCore::LocalFrameViewLayoutContext::layout() > 18 0x2847cca74 > WebCore::LocalFrameView::updateLayoutAndStyleIfNeededRecursive() > 19 0x284832188 WebCore::Page::layoutIfNeeded() > 20 0x284832ca4 WebCore::Page::updateRendering() > 21 0x11a3a5aac WebKit::WebPage::updateRendering() > 22 0x119443dc8 > WebKit::TiledCoreAnimationDrawingArea::updateRendering(WebKit:: > TiledCoreAnimationDrawingArea::UpdateRenderingType) > 23 0x119443cc8 WebKit::TiledCoreAnimationDrawingArea::forceRepaint() > 24 0x11a3a462c WebKit::WebPage::forceRepaintWithoutCallback() > 25 0x119cbec04 WKBundlePageForceRepaint > 26 0x10184959c WTR::InjectedBundlePage::dump() > 27 0x10184d988 WTR::dumpAfterWaitAttributeIsRemoved(OpaqueWKBundlePage > const*) > 28 0x101848de4 > WTR::InjectedBundlePage::frameDidChangeLocation(OpaqueWKBundleFrame const*) > 29 0x101847250 > WTR::InjectedBundlePage::didFinishLoadForFrame(OpaqueWKBundleFrame const*) > 30 0x101845e1c > WTR::InjectedBundlePage::didFinishLoadForFrame(OpaqueWKBundlePage const*, > OpaqueWKBundleFrame const*, void const**, void const*) > 31 0x119cafa18 > WebKit::InjectedBundlePageLoaderClient::didFinishLoadForFrame(WebKit:: > WebPage&, WebKit::WebFrame&, WTF::RefPtr<API::Object, > WTF::RawPtrTraits<API::Object>, WTF::DefaultRefDerefTraits<API::Object>>&) > com.apple.WebKit.WebContent.Development terminated (pid 5864) for reason: > crash > LEAK: 1 WebPageProxy Oops, this assertion is stale. I'll remove it. 1986268 6 ysuzuki 2023-10-19 14:20:33 -0700 https://github.com/WebKit/WebKit/commit/cd9fc42579dc0b7307a9c1bb3f48a2115f1443c8 Fixed in this. It is a bit strange that the original patch was landed with safe-merge-queue :( 1986786 7 mark.lam 2023-10-23 00:41:41 -0700 *** Bug 262409 has been marked as a duplicate of this bug. *** 2054363 8 fujii 2024-08-21 00:29:12 -0700 Regressed: Bug 277624 – REGRESSION (269500@main Safari 17.5): Masked Curved Line Rendering Issue