264625 2023-11-10 15:05:00 -0800 RenderBox::applyVisualEffectOverflow() and RenderBox::outlineBoundsForRepaint() use different ways of computing outline/shadow extent 2023-11-28 18:35:27 -0800 1 1 1 Unclassified WebKit Layout and Rendering WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=258740 InRadar P2 Normal --- 1 simon.fraser simon.fraser ahmad.saleem792 bfulgham simon.fraser webkit-bug-importer zalan oldest_to_newest 1991703 0 simon.fraser 2023-11-10 15:05:00 -0800 RenderBox::applyVisualEffectOverflow() and RenderBox::outlineBoundsForRepaint() have different logic for combining outline/shadow extents, and give different answers on an element like: .box { margin: 30px; width: 50%; height: 500px; border: 12px solid gray; outline: 5px solid green; box-shadow: 0 0 0px 20px black; } 1991704 1 simon.fraser 2023-11-10 15:05:25 -0800 RenderBox 0x138003430 outlineBoundsForRepaint - outline bounds (-25,-25) width=225 height=574 bigger than visual overflow (-20,-20) width=215 height=564 1991707 2 simon.fraser 2023-11-10 15:11:35 -0800 These should give the same result: applyVisualEffectOverflow(borderBoxRect()) and LayoutRect box = borderBoundingBox(); adjustRectForOutlineAndShadow(box); 1993589 3 webkit-bug-importer 2023-11-17 15:05:13 -0800 <rdar://problem/118582062> 1995114 4 simon.fraser 2023-11-27 16:20:38 -0800 Pull request: https://github.com/WebKit/WebKit/pull/20961 1995499 5 ews-feeder 2023-11-28 18:35:25 -0800 Committed 271254@main (6aa1f6767674): <https://commits.webkit.org/271254@main> Reviewed commits have been landed. Closing PR #20961 and removing active labels.