93772 2012-08-11 18:18:51 -0700 SVG as image drawn differently after layout 2022-07-15 15:27:57 -0700 1 1 1 Unclassified WebKit SVG 420+ Unspecified Unspecified RESOLVED CONFIGURATION CHANGED P2 Normal --- 94377 1 pdr pdr bfulgham dtrebbien zimmermann oldest_to_newest 693159 0 157882 pdr 2012-08-11 18:18:51 -0700 Created attachment 157882 Testcase (1 of 2) If we have an SVG file referenced as an image, the image will draw differently depending on if a layout has occurred: image.onload = function() { var context1 = canvas1.getContext('2d'); context1.drawImage(img, 0, 0); // draws correctly setTimeout(function(){ var context2 = canvas2.getContext('2d'); context2.drawImage(img, 0, 0); // draws incorrectly, scaled 2x? }, 1); } My guess is we're changing the size of the SVG during layout. This could be the source of a bug I've seen sporadically where SVG embedded in an image renders incorrectly on first load. 693160 1 157883 pdr 2012-08-11 18:19:19 -0700 Created attachment 157883 Testcase (2 of 2) 694545 2 pdr 2012-08-13 23:18:47 -0700 Just updating to report what I've found so far. I think there is more than one bug hiding in here :( I think one bug is that we use the device scale factor in calculating the bitmap size in SVGImageCache.cpp. It looks like implementations have steered towards doing that one level deeper (see http://code.google.com/p/chromium/issues/detail?id=127852). This isn't surprising since this code was written long ago. A second bug is that on the first drawImage() call we have not done a layout of the SVG image, so it does not have size and scales set, and so we end up using drawImage directly. On the second drawImage() we have done a layout so we have cached data, but the cached data has zoom applied. The second drawImage therefore has zoom applied twice. This second bug may get more complex because as a part of fixing our SVG sizing issues, it looks like a layout may be needed just to determine the aspect ratio of SVG content. 1884299 3 bfulgham 2022-07-15 15:27:57 -0700 Safari, Chrome, and Firefox all agree on rendering for this test case. I don't believe there is any remaining compatibility issue. 157882 2012-08-11 18:18:51 -0700 2012-08-11 18:18:51 -0700 Testcase (1 of 2) bug.html text/html 1291 pdr PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KICAgIDxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+ CiAgICAgICAgLnNjcm9sbCB7CiAgICAgICAgICAgIHBvc2l0aW9uOiBhYnNvbHV0ZTsKICAgICAg ICAgICAgb3ZlcmZsb3c6IGhpZGRlbjsKICAgICAgICAgICAgd2lkdGg6IDIyMHB4OwogICAgICAg ICAgICBoZWlnaHQ6IDIyMHB4OwogICAgICAgICAgICB0b3A6IDMwcHg7CiAgICAgICAgICAgIGJv cmRlcjogMXB4IHNvbGlkIGJsYWNrOwogICAgICAgIH0KICAgICAgICAuc2Nyb2xsI29uZSB7CiAg ICAgICAgICAgIGxlZnQ6IDMwcHg7CiAgICAgICAgfQogICAgICAgIC5zY3JvbGwjdHdvIHsKICAg ICAgICAgICAgbGVmdDogMzAwcHg7CiAgICAgICAgfQogICAgICAgIC5zY3JvbGwjdGhyZWUgewog ICAgICAgICAgICBsZWZ0OiA1NzBweDsKICAgICAgICB9CiAgICA8L3N0eWxlPgo8L2hlYWQ+Cjxi b2R5PgogICAgPGRpdiBjbGFzcz0ic2Nyb2xsIiBpZD0ib25lIj4KICAgICAgICA8aW1nIGlkPSJp bWciIHNyYz0ic3ZnLnN2ZyI+PC9pbWc+CiAgICA8L2Rpdj4KICAgIDxkaXYgY2xhc3M9InNjcm9s bCIgaWQ9InR3byI+CiAgICAgICAgPGNhbnZhcyBpZD0iY2FudmFzMSIgd2lkdGg9IjIwMCIgaGVp Z2h0PSIyMDAiPjwvY2FudmFzPgogICAgPC9kaXY+CiAgICA8ZGl2IGNsYXNzPSJzY3JvbGwiIGlk PSJ0aHJlZSI+CiAgICAgICAgPGNhbnZhcyBpZD0iY2FudmFzMiIgd2lkdGg9IjIwMCIgaGVpZ2h0 PSIyMDAiPjwvY2FudmFzPgogICAgPC9kaXY+CgogICAgPHNjcmlwdD4KICAgICAgICB2YXIgaW1n U291cmNlID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2ltZycpOwogICAgICAgIHZhciBjYW52 YXMxID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2NhbnZhczEnKTsKICAgICAgICB2YXIgY2Fu dmFzMiA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdjYW52YXMyJyk7CgogICAgICAgIGltZ1Nv dXJjZS5vbmxvYWQgPSBmdW5jdGlvbigpIHsKICAgICAgICAgICAgdmFyIGNvbnRleHQxID0gY2Fu dmFzMS5nZXRDb250ZXh0KCcyZCcpOwogICAgICAgICAgICBjb250ZXh0MS5kcmF3SW1hZ2UoaW1n LCAwLCAwKTsKCiAgICAgICAgICAgIHNldFRpbWVvdXQoZnVuY3Rpb24oKXsKICAgICAgICAgICAg ICAgIHZhciBjb250ZXh0MiA9IGNhbnZhczIuZ2V0Q29udGV4dCgnMmQnKTsKICAgICAgICAgICAg ICAgIGNvbnRleHQyLmRyYXdJbWFnZShpbWcsIDAsIDApOwogICAgICAgICAgICB9LCAxKTsKICAg ICAgICB9CiAgICA8L3NjcmlwdD4KPC9ib2R5Pgo8L2h0bWw+Cg== 157883 2012-08-11 18:19:19 -0700 2012-08-11 18:19:19 -0700 Testcase (2 of 2) svg.svg image/svg+xml 188 pdr PHN2ZyBpZD0ic3ZnIiB3aWR0aD0iMjAwIiBoZWlnaHQ9IjIwMCIgeG1sbnM9Imh0dHA6Ly93d3cu dzMub3JnLzIwMDAvc3ZnIiB4bWxuczp4bGluaz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94bGlu ayI+CiAgICAgIDxyZWN0IHg9IjAiIHk9IjAiIHdpZHRoPSIxMDAiIGhlaWdodD0iMTAwIiBmaWxs PSJncmVlbiIvPgo8L3N2Zz4=