[empathy/gnome-2-34: 3/7] Ensure our bin doesn't change state before being in the hash table



commit e026781c4d7974cc047a823512fb6cb8aa266f17
Author: Sjoerd Simons <sjoerd simons collabora co uk>
Date:   Mon Mar 14 09:37:43 2011 +0000

    Ensure our bin doesn't change state before being in the hash table

 src/empathy-audio-sink.c |   12 ++++++++++--
 1 files changed, 10 insertions(+), 2 deletions(-)
---
diff --git a/src/empathy-audio-sink.c b/src/empathy-audio-sink.c
index 31fffff..1120dae 100644
--- a/src/empathy-audio-sink.c
+++ b/src/empathy-audio-sink.c
@@ -357,15 +357,23 @@ empathy_audio_sink_request_new_pad (GstElement *element,
   if (!gst_element_add_pad (GST_ELEMENT (bin), subpad))
     goto error;
 
+
+  /* Ensure that state changes only happen _after_ the element has been added
+   * to the hash table. But add it to the bin first so we can create our
+   * ghostpad (if we create the ghostpad before adding it to the bin it will
+   * get unlinked) */
+  gst_element_set_locked_state (GST_ELEMENT (bin), TRUE);
   gst_bin_add (GST_BIN (self), bin);
 
-  /* Add elemnt into the hash table before syncing state so we know about it
-   * when elements are added */
   pad = gst_ghost_pad_new (name, subpad);
+  g_assert (pad != NULL);
+
   audiobin = audio_bin_new (pad, bin, volume, sink);
 
   g_hash_table_insert (self->priv->audio_bins, pad, audiobin);
 
+  gst_element_set_locked_state (GST_ELEMENT (bin), FALSE);
+
   if (!gst_element_sync_state_with_parent (bin))
     goto error;
 



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]