Fix abs_bounding_box calculation for image (#924)

Author: Dabble63

crates/resvg/examples/bboxes.svg

@@ -234,7 +234,7 @@ pub(crate) fn convert_inner(
     );
 
     let abs_transform = parent.abs_transform.pre_concat(image_ts);
-    let abs_bounding_box = rect.transform(abs_transform)?;
+    let abs_bounding_box = view_box.transform(parent.abs_transform)?;
 
     let mut g = Group::empty();
     g.id = id;

crates/usvg/src/parser/image.rs

@@ -1,6 +1,7 @@
 // Copyright 2018 the Resvg Authors
 // SPDX-License-Identifier: Apache-2.0 OR MIT
 
+use tiny_skia_path::Rect;
 use usvg::Color;
 
 #[test]
@@ -502,3 +503,33 @@ fn svg_without_xmlns() {
     let tree = usvg::Tree::from_str(&svg, &usvg::Options::default()).unwrap();
     assert_eq!(tree.size(), usvg::Size::from_wh(100.0, 100.0).unwrap());
 }
+
+#[test]
+fn image_bbox_with_parent_transform() {
+    let svg = "
+    <svg viewBox='0 0 200 200' 
+         xmlns='http://www.w3.org/2000/svg'
+         xmlns:xlink='http://www.w3.org/1999/xlink'>
+        <g transform='translate(25 25)'>
+            <image id='image1' x='10' y='10' width='50' height='50' xlink:href='data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAGQAAABkCAYAAABw4pVUAAABb0lEQVR4Xu3VUQ0AIAzEUOZfA87wAgkq+vGmoGlz2exz73IZAyNIpsUHEaTVQ5BYD0EEqRmI8fghgsQMxHAsRJCYgRiOhQgSMxDDsRBBYgZiOBYiSMxADMdCBIkZiOFYiCAxAzEcCxEkZiCGYyGCxAzEcCxEkJiBGI6FCBIzEMOxEEFiBmI4FiJIzEAMx0IEiRmI4ViIIDEDMRwLESRmIIZjIYLEDMRwLESQmIEYjoUIEjMQw7EQQWIGYjgWIkjMQAzHQgSJGYjhWIggMQMxHAsRJGYghmMhgsQMxHAsRJCYgRiOhQgSMxDDsRBBYgZiOBYiSMxADMdCBIkZiOFYiCAxAzEcCxEkZiCGYyGCxAzEcCxEkJiBGI6FCBIzEMOxEEFiBmI4FiJIzEAMx0IEiRmI4ViIIDEDMRwLESRmIIZjIYLEDMRwLESQmIEYjoUIEjMQw7EQQWIGYjgWIkjMQAzHQgSJGYjhWIggMQMxnAdKSlrwlejIDgAAAABJRU5ErkJggg=='/>
+        </g>
+    </svg>
+    ";
+
+    let tree = usvg::Tree::from_str(&svg, &usvg::Options::default()).unwrap();
+
+    let usvg::Node::Group(group_node1) = &tree.root().children()[0] else {
+        unreachable!()
+    };
+    let usvg::Node::Group(group_node2) = &group_node1.children()[0] else {
+        unreachable!()
+    };
+    let usvg::Node::Image(image_node) = &group_node2.children()[0] else {
+        unreachable!()
+    };
+
+    assert_eq!(
+        image_node.abs_bounding_box(),
+        Rect::from_xywh(35.0, 35.0, 50.0, 50.0).unwrap()
+    );
+}