decodebin: Properly deactivate ghostpads
authorEdward Hervey <edward@centricular.com>
Fri, 6 Nov 2015 18:31:47 +0000 (19:31 +0100)
committerEdward Hervey <bilboed@bilboed.com>
Fri, 6 Nov 2015 18:38:13 +0000 (19:38 +0100)
commitd0eface01c9af0ab1d61fe4916ca945ae0679af7
tree836abc1f3ecc970636cbbee5f332ae43553a28f3
parentd2e210bbeabc2949740803ba4daa9265662e50d0
decodebin: Properly deactivate ghostpads

Just setting the ghostpad as flushing wasn't enough. It needs to be
consistent on the internal proxypad also, otherwise you end up in
situations where:
* a pending buffer on the target pad triggers the sticky event
  propagation
* the default implementation sees that the proxypad is not flushing,
  so it tries to push it to the other pad (the actual ghostpad)
* the ghostpad is flushing, so returns FALSE
* the push_event function sees that pushing the event failed...
* ... and pending buffer push returns GST_FLOW_ERROR, instead of
  GST_FLOW_FLUSHING

By using gst_pad_set_active(FALSE), we ensure that both the ghostpad
and the proxypad are flushing/deactivated. The situation above will
no longer occur, and a GST_FLOW_FLUSHING will be returned.
gst/playback/gstdecodebin2.c