GNOME Bugzilla – Bug 369134
video disappears when moving totem to another screen
Last modified: 2012-10-09 16:19:03 UTC
Steps to repro: 0) Xnest -ac -scrns 2 :1 1) Apply the attached patch to totem 2) $ DISPLAY=:1 totem 3) Load a video 4) Help->Move to next screen Result: Video disappears. When you move the window back to the original screen, it doesn't come back either. This is with totem/gst (cvs); totem/xine crashes when moving the window to the 2nd screen.
Created attachment 75807 [details] add menu item for moving totem window to another screen
Neat. My guess is that the backends are stupid as sin handling this. I'm pretty sure that the xine-lib backend would need a complete tear-down, and build-up to be able to handle this. Not sure about the GStreamer one.
This is obviously a backend related matter, so cloning and reassigning.
Created attachment 77440 [details] patch for gst video sinks
Created attachment 77441 [details] [review] patch for bvw So I patched the gst x(v)imagesink to use a different screen, and made bvw use that; but when I move screens, it crashes in some other thread updating the output on the _old_ display.... am I missing some locks maybe, or is this inherently impossible? (I tested only ximagesink, since Xnest doesn't support xv.)
Created attachment 211248 [details] [review] Add "Move to next screen" menu item
Comment on attachment 75807 [details] add menu item for moving totem window to another screen Obsoleted by the next uploaded patch
Christian, do you have instructions on how to reproduce the bug with Xephyr? Xnets just crashes for me.
AFAIK Xephyr doesn't do multi-screen.
Review of attachment 211248 [details] [review]: ::: data/totem.ui @@ +263,3 @@ + <child> + <object class="GtkAction" id="move-to-screen"> + <property name="label" translatable="yes">Move to next screen</property> Needs to be Title Case. Because Everybody Loves Arbitrary Capitalisation. ::: src/totem-menu.c @@ +1392,3 @@ + } + else + { Shouldn't the braces be inline with the ‘else’?
(In reply to comment #10) > Review of attachment 211248 [details] [review]: It's a test patch, we're not going to commit that...
I'll say that we'll care about this when we have a way of reproducing the problem.