236344 2022-02-08 18:48:17 -0800 SVG getEnclosureList is effected by page zoom. 2022-02-14 19:58:23 -0800 1 1 1 Unclassified WebKit SVG Safari 15 Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=224795 InRadar P2 Normal --- 1 halifirien webkit-unassigned halifirien sabouhallawa simon.fraser webkit-bug-importer wenson_hsieh zalan zimmermann oldest_to_newest 1838982 0 451334 halifirien 2022-02-08 18:48:17 -0800 Created attachment 451334 demo of issue SVG getEnclosureList is effected by page zoom. Attached is an html file with an svg element with 2 rectangles. The large rectangle entirely encloses a small rectangle. Calling `getBBox` on the large rectangle returns an SVGRect (box). Calling svg.getEnclosureList(box, svg) correctly returns the smaller rect. Now when you use ⌘+ to Page zoom, getEnclosureList no longer returns any elements. this effects safari 15, but I can confirm it also effect safari 13 (could not test 14) thank you. 1839121 1 halifirien 2022-02-09 06:54:23 -0800 chromium keeps svg.currentScale at 1; even at different page zoom levels. While webkits svg.currentScale tracks the page zoom. however, you can manually set svg.currentScale - and adjusting it has no effect on svg.getEnclosureList on chromium. 1839158 2 halifirien 2022-02-09 08:08:52 -0800 getEnclosureList (from svg/SVGSVGElement.cpp) appears to route to LegacyRenderSVGModelObject::checkEnclosure, even though a (modern?) RenderSVGModelObject::checkEnclosure exists. 1839205 3 halifirien 2022-02-09 09:09:55 -0800 ah, I see that RenderSVGModelObject::checkEnclosure is very new (https://trac.webkit.org/changeset/287538/webkit) - and not even used anywhere yet 1839340 4 halifirien 2022-02-09 12:11:21 -0800 maybe related to https://bugs.webkit.org/show_bug.cgi?id=194903 1841561 5 webkit-bug-importer 2022-02-14 19:58:23 -0800 <rdar://problem/88943590> 451334 2022-02-08 18:48:17 -0800 2022-02-08 18:48:17 -0800 demo of issue svg.html text/html 1158 halifirien PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KICA8aGVhZD4KICAgIDx0aXRsZT5zdmcg Z2V0RW5jbG9zdXJlTGlzdDwvdGl0bGU+CiAgPC9oZWFkPgogIDxzY3JpcHQgdHlwZT0ibW9kdWxl Ij4KICAgIGxldCBidXR0b24gPSBkb2N1bWVudC5xdWVyeVNlbGVjdG9yKCdidXR0b24nKQogICAg bGV0IHN2ZyA9ICBkb2N1bWVudC5xdWVyeVNlbGVjdG9yKCdzdmcnKQogICAgYnV0dG9uLnRleHRD b250ZW50ID0gYHRlc3QgZ2V0RW5jbG9zdXJlTGlzdFxuc3ZnIGN1cnJlbnRTY2FsZTogJHtzdmcu Y3VycmVudFNjYWxlfWAKICAgIGJ1dHRvbi5hZGRFdmVudExpc3RlbmVyKCdjbGljaycsIChlKSA9 PiB7CiAgICAgIGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3IoJyNzbWFsbCcpLnN0eWxlLmZpbGwgPSAn Jy8vY2xlYXIgZmlsbAogICAgICBsZXQgcmVjdCA9IGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3IoJyNs YXNzbycpCiAgICAgIGxldCBib3ggPSByZWN0LmdldEJCb3goKQogICAgICBjb25zb2xlLmxvZyhi b3gpCiAgICAgIGZvciAobGV0IGVsIG9mIHN2Zy5nZXRFbmNsb3N1cmVMaXN0KGJveCwgc3ZnKSkg ewogICAgICAgIGVsLnN0eWxlLmZpbGwgPSAicmVkIgogICAgICB9CiAgICB9KQogICAgd2luZG93 LmFkZEV2ZW50TGlzdGVuZXIoJ3Jlc2l6ZScsIChlKSA9PiB7CiAgICAgIGJ1dHRvbi50ZXh0Q29u dGVudCA9IGB0ZXN0IGdldEVuY2xvc3VyZUxpc3RcbnN2ZyBjdXJyZW50U2NhbGU6ICR7c3ZnLmN1 cnJlbnRTY2FsZX1gCiAgICB9KTsKICA8L3NjcmlwdD4KICA8Ym9keT4KICAgIDxzdmcgd2lkdGg9 IjYwMCIgaGVpZ2h0PSc1MDAnIHZpZXdCb3g9IjAgMCA2MDAgNTAwIiBzdHlsZT0iYm9yZGVyOiB0 aGluIGJsYWNrIHNvbGlkIj4KICAgICAgPHJlY3QgaWQ9Imxhc3NvIiB4PSIxMDAiIHk9IjEwMCIg d2lkdGg9IjMwMCIgaGVpZ2h0PSIzMDAiIGZpbGw9Im5vbmUiIHN0cm9rZT0iYmxhY2siIHN0cm9r ZS1kYXNoYXJyYXk9IjUgNSIvPgogICAgICA8cmVjdCBpZD0ic21hbGwiIHg9IjI5MCIgeT0iMjkw IiB3aWR0aD0iMTAwIiBoZWlnaHQ9IjEwMCIgZmlsbD0ibm9uZSIgc3Ryb2tlPSJibGFjayIvPgog ICAgPC9zdmc+CiAgICA8YnV0dG9uIHN0eWxlPSJ3aGl0ZS1zcGFjZTogcHJlOyI+PC9idXR0b24+ CiAgPC9ib2R5Pgo8L2h0bWw+