camerabin: remove custom renegotiate event
authorThiago Santos <thiago.sousa.santos@collabora.com>
Tue, 1 May 2012 18:29:37 +0000 (15:29 -0300)
committerThiago Santos <thiago.sousa.santos@collabora.com>
Tue, 1 May 2012 18:31:11 +0000 (15:31 -0300)
In 1.0 there is a core reconfigure event, camerabin doesn't need
its custom renegotiate event anymore

gst/camerabin2/camerabin2-src.txt
gst/camerabin2/gstcamerabin2.c

index ef031f98067578736d85320335d970ece86329e0..f50288e674664fd42fd077786ceee91c294365f5 100644 (file)
@@ -71,11 +71,3 @@ caps negotiation. Camerabin2 puts capsfilters downstream from each of the
 camera source pads. The camera source can simply get_caps on the peer of
 each of its pads to know what are the allowed caps for that pad.
 
-
--- Renegotiation --
-Easy renegotiation isn't supported on gstreamer yet (there is some ongoing work
-currently). Camerabin2 will use a custom 'renegotiate' event, no fields to
-indicate that a certain pad should renegotiate its caps.
-
-Upon receiving this event, a pad should get_caps on its peer and do the caps
-negotiation again. It is likely that a new format was requested.
index 93f0185189daa99a87f32994cc667492a2d11577..344aa192599c7e22eaf6b03a61aa713585540ff5 100644 (file)
@@ -347,13 +347,6 @@ gst_camera_bin_change_state (GstElement * element, GstStateChange trans);
 
 /* Camerabin functions */
 
-static GstEvent *
-gst_camera_bin_new_event_renegotiate (void)
-{
-  return gst_event_new_custom (GST_EVENT_CUSTOM_BOTH,
-      gst_structure_new_empty ("renegotiate"));
-}
-
 static GstEvent *
 gst_camera_bin_new_event_file_location (const gchar * location)
 {
@@ -2094,13 +2087,8 @@ gst_camera_bin_set_property (GObject * object, guint prop_id,
         GST_WARNING_OBJECT (camera, "Image capsfilter missing");
       }
 
-      /* set the capsfilter caps and notify the src to renegotiate */
-      if (pad) {
-        GST_DEBUG_OBJECT (camera, "Pushing renegotiate on %s",
-            GST_PAD_NAME (pad));
-        gst_pad_send_event (pad, gst_camera_bin_new_event_renegotiate ());
+      if (pad)
         gst_object_unref (pad);
-      }
     }
       break;
     case PROP_VIDEO_CAPTURE_CAPS:{
@@ -2115,7 +2103,6 @@ gst_camera_bin_set_property (GObject * object, guint prop_id,
           "Setting video capture caps to %" GST_PTR_FORMAT,
           gst_value_get_caps (value));
 
-      /* set the capsfilter caps and notify the src to renegotiate */
       if (G_LIKELY (camera->videobin_capsfilter)) {
         g_object_set (camera->videobin_capsfilter, "caps",
             gst_value_get_caps (value), NULL);
@@ -2124,9 +2111,6 @@ gst_camera_bin_set_property (GObject * object, guint prop_id,
       }
 
       if (pad) {
-        GST_DEBUG_OBJECT (camera, "Pushing renegotiate on %s",
-            GST_PAD_NAME (pad));
-        gst_pad_send_event (pad, gst_camera_bin_new_event_renegotiate ());
         gst_object_unref (pad);
       }
     }
@@ -2143,7 +2127,6 @@ gst_camera_bin_set_property (GObject * object, guint prop_id,
           "Setting viewfinder capture caps to %" GST_PTR_FORMAT,
           gst_value_get_caps (value));
 
-      /* set the capsfilter caps and notify the src to renegotiate */
       if (G_LIKELY (camera->viewfinderbin_capsfilter)) {
         g_object_set (camera->viewfinderbin_capsfilter, "caps",
             gst_value_get_caps (value), NULL);
@@ -2152,9 +2135,6 @@ gst_camera_bin_set_property (GObject * object, guint prop_id,
       }
 
       if (pad) {
-        GST_DEBUG_OBJECT (camera, "Pushing renegotiate on %s",
-            GST_PAD_NAME (pad));
-        gst_pad_send_event (pad, gst_camera_bin_new_event_renegotiate ());
         gst_object_unref (pad);
       }
     }