#7 gmplayer: fullscreen problem

[Anonymous]

software: icewm-1.2.14pre10, mplayer (1.0pre3 or cvs).

play any videos in gmplayer, use fullscreen, click on
the video: the screen is black!

other wm are ok, but I like only icewm!

[Adam Pribyl]

Logged In: YES
user_id=607018

I can confirm this. When you click again with visible cursor
into black screen, the fullscreen picture appears again.

This is only problem with gmplayer = graphical gui. This
does not happen with mplayer itself.

I think this is related to what gmplayer should do when
clicked into the picture. Probably it should hide gui but it
switches somewhere else.

[Marko Macek]

Logged In: YES
user_id=1814

Hmm, for me gmplayer (1.0pre2) opens 2 windows (1 blank +
unresponsive control window) and a normal mplayer that
works. Then switching to fullscreen and back kills it. Must
be a bug?

[Nobody/Anonymous]

Logged In: NO

yes, only in GUI. but in wmaker and in fluxbox it's ok. is
it a bug of mplayer o icewm? if it's a bug of mplayer, close
this one and I'll inform mplayer team. Thanks.

[Marko Macek]

Logged In: YES
user_id=1814

I am now able to reproduce the problem.

It happens when I click twice on the video.

The video comes back when I right click.

[Marko Macek]

Logged In: YES
user_id=1814

After a few hours debugging and looking at mplayer sources I
only found one thing:

doing an 'xrefresh' (from remote machine) seems to fix the
problem. It looks like an Expose event being mishandled or
something. Still not sure if the bug is in icewm or mplayer.

[Marko Macek]

Logged In: YES
user_id=1814

considered duplicate of 799320.

[Marko Macek]

Logged In: YES
user_id=1814

hmm, no (was something else).

It seems to me that the bug is in mplayer.

The problem is in line: (libvo/vo_xv.c)

static void check_events(void)
{
int e = vo_x11_check_events(mDisplay);

if (e & VO_EVENT_EXPOSE && vo_fs) {
-> vo_x11_clearwindow(mDisplay, vo_window);

I am not 100% sure what is happening, but there seems to be
a race condition with:

const Atom autopaint =
XInternAtom(mDisplay,
"XV_AUTOPAINT_COLORKEY", False);
XvSetPortAttribute(mDisplay, xv_port,
autopaint, 1);

[Marko Macek]

Logged In: YES
user_id=1814

possible mplayer fix attached

[Marko Macek]

fix for mplayer black screen problem (v1)

[Marko Macek]

Logged In: YES
user_id=1814

current mplayer CVS has this fixed for some time
(due to efforts of beastd and me).