faceoverlay: don't make rsvgoverlay reload the SVG for every single frame

Only set location on rsvgoverlay if it has changed.

diff --git a/gst/faceoverlay/gstfaceoverlay.c b/gst/faceoverlay/gstfaceoverlay.c
index b3442b2bc2924b89e98250a7c029921fda41665c..da3cb40705c972b2cd852c7e8e1063aded3c639a 100644 (file)
--- a/gst/faceoverlay/gstfaceoverlay.c
+++ b/gst/faceoverlay/gstfaceoverlay.c
@@ -174,6 +174,7 @@ gst_face_overlay_change_state (GstElement * element, GstStateChange transition)
                 "facedetect element or rsvgoverlay"));
         return GST_STATE_CHANGE_FAILURE;
       }
+      filter->update_svg = TRUE;
       break;
     default:
       break;
@@ -237,11 +238,16 @@ gst_face_overlay_handle_faces (GstFaceOverlay * filter, GstStructure * s)
   svg_width = (int) width *filter->w;
   svg_height = (int) height *filter->h;
 
-  GST_LOG_OBJECT (filter, "setting overlay to %d x %d @ %d,%d (file: %s)",
-      svg_width, svg_height, svg_x, svg_y, GST_STR_NULL (filter->location));
+  if (filter->update_svg) {
+    GST_DEBUG_OBJECT (filter, "set rsvgoverlay location=%s", filter->location);
+    g_object_set (filter->svg_overlay, "location", filter->location, NULL);
+    filter->update_svg = FALSE;
+  }
+
+  GST_LOG_OBJECT (filter, "overlay dimensions: %d x %d @ %d,%d",
+      svg_width, svg_height, svg_x, svg_y);
 
   g_object_set (filter->svg_overlay,
-      "location", filter->location,
       "x", svg_x, "y", svg_y, "width", svg_width, "height", svg_height, NULL);
 }
 
@@ -347,6 +353,7 @@ gst_face_overlay_set_property (GObject * object, guint prop_id,
   switch (prop_id) {
     case PROP_LOCATION:
       filter->location = g_value_dup_string (value);
+      filter->update_svg = TRUE;
       break;
     case PROP_X:
       filter->x = g_value_get_float (value);

diff --git a/gst/faceoverlay/gstfaceoverlay.h b/gst/faceoverlay/gstfaceoverlay.h
index 7f4427c28e039736ca4787c395545937f4836fc0..c21327b11a3583de3c54b5b1ffac2d9dfb720429 100644 (file)
--- a/gst/faceoverlay/gstfaceoverlay.h
+++ b/gst/faceoverlay/gstfaceoverlay.h
@@ -73,6 +73,8 @@ struct _GstFaceOverlay
 
   gboolean process_message;
 
+  gboolean update_svg;
+
   gchar *location;
   gfloat x;
   gfloat y;