252903 2023-02-24 07:30:40 -0800 [Resize-observer] The `getBoundingBox` of a non-empty SVG element is 0x0 2023-03-27 06:24:04 -0700 1 1 1 Unclassified WebKit CSS WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 cathiechen cathiechen cathiechen rbuis sabouhallawa simon.fraser webkit-bug-importer zalan oldest_to_newest 1936516 0 465152 cathiechen 2023-02-24 07:30:40 -0800 Created attachment 465152 test-case In ResizeObservation::computeObservedSizes(), it calls `downcast<SVGElement>(*m_target).getBoundingBox()` which returns 0x0 when the layout size of SVG is not empty. See the attached test. 1936531 1 cathiechen 2023-02-24 08:50:12 -0800 According to [1], > Web content can also contain SVG elements. SVG elements that do not have associated CSS layout boxes define bounding box instead of a content box. Content rect for the SVGGraphicsElements without CSS layout boxes is a rect whose: > width is bounding box width > height is bounding box height So from ResizeObserver size, it is correct to call `downcast<SVGElement>(*m_target).getBoundingBox()` in `ResizeObservation::computeObservedSizes()`. I'll leave this bug to an SVG Expert. [1] https://drafts.csswg.org/resize-observer/ 1938262 2 cathiechen 2023-03-02 13:28:54 -0800 Pull request: https://github.com/WebKit/WebKit/pull/10955 1938500 3 webkit-bug-importer 2023-03-03 07:31:14 -0800 <rdar://problem/106194536> 1944202 4 ews-feeder 2023-03-27 06:24:00 -0700 Committed 262145@main (afa718b1cfd9): <https://commits.webkit.org/262145@main> Reviewed commits have been landed. Closing PR #10955 and removing active labels. 465152 2023-02-24 07:30:40 -0800 2023-02-24 07:30:40 -0800 test-case auto-007-real-simplify-003.html text/html 785 cathiechen PCFET0NUWVBFIGh0bWw+CjxzdHlsZT4KICAgIC50ZXN0IHsKICAgICAgICB3aWR0aDogbWF4LWNv bnRlbnQ7CiAgICAgICAgaGVpZ2h0OiBtYXgtY29udGVudDsKICAgICAgICBib3JkZXI6IDFweCBz b2xpZDsKICAgIH0KCiAgICAudGVzdDo6YmVmb3JlIHsKICAgICAgICBjb250ZW50OiAiIjsKICAg ICAgICBkaXNwbGF5OiBibG9jazsKICAgICAgICB3aWR0aDogMzIwcHg7CiAgICAgICAgaGVpZ2h0 OiAyNDBweDsKICAgIH0KPC9zdHlsZT4KCjxkaXYgaWQ9ImxvZyI+PC9kaXY+CjxkaXYgaWQ9InRl c3RzIj4KICAgIDxzdmcgaWQ9InRhcmdldCIgY2xhc3M9InRlc3QiPjwvc3ZnPgo8L2Rpdj4KCgo8 c2NyaXB0Pgp2YXIgcm8gPSBuZXcgUmVzaXplT2JzZXJ2ZXIoZW50cmllcyA9PiB7CiAgICBmb3Ig KGxldCBlbnRyeSBvZiBlbnRyaWVzKSB7CiAgICAgICAgY29uc3QgY3IgPSBlbnRyeS5jb250ZW50 UmVjdDsKICAgICAgICB2YXIgdGV4dCA9ICdFbGVtZW50OicgKyBlbnRyeS50YXJnZXQgKyAnXG4n OwogICAgICAgIHRleHQgPSB0ZXh0ICsgYEVsZW1lbnQgc2l6ZTogJHtjci53aWR0aH1weCB4ICR7 Y3IuaGVpZ2h0fXB4YCArICdcbic7CiAgICAgICAgdGV4dCA9IHRleHQgKyBgRWxlbWVudCBwYWRk aW5nOiAke2NyLnRvcH1weCA7ICR7Y3IubGVmdH1weGA7CiAgICAgICAgZG9jdW1lbnQucXVlcnlT ZWxlY3RvcignI2xvZycpLmlubmVyVGV4dCA9IHRleHQ7CiAgICB9Cn0pOwpyby5vYnNlcnZlKGRv Y3VtZW50LnF1ZXJ5U2VsZWN0b3IoJyN0YXJnZXQnKSk7Cjwvc2NyaXB0Pgo=