clip: make remove_child a reverse of add_child
authorHenry Wilkes <hwilkes@igalia.com>
Mon, 2 Mar 2020 13:35:20 +0000 (13:35 +0000)
committerHenry Wilkes <hwilkes@igalia.com>
Mon, 16 Mar 2020 14:19:52 +0000 (14:19 +0000)
commit3af38e171942b991e1a4e962ab5895b3d63980cc
tree5f8d4b7bf8e60722ec65aa2de2da19f5bafbc3cb
parentd1276ed29b05ecb490a2c7093fcd2146a023330e
clip: make remove_child a reverse of add_child

Previously, we relied on ->child_removed to reverse the priority changes
that occured in ->add_child. However, ->child_removed is not always
called (the signal child-removed is not always emitted) when a
->add_child needs to be removed. However, ->remove_child is always
called to reverse ->add_child, so the code was moved here. Otherwise, we
risk that the priorities of the clip will contain gaps, which will cause
problems when another child is added to the clip.
ges/ges-clip.c