mplayer.hg: gui/interface.c comparison

comparison gui/interface.c @ 33960:1cd81338af07

Rework fullscreen handling. This should fix all problems the GUI had with fullscreen handling. In interface.c: 1. Remove disabled code. 2. Set background of inactive video window to black (which prevents some flashing when starting the playback in fullscreen mode). 3. Set the state of the fullscreen button according to next playback mode (normal or fullscreen). 4. Make the video window invisible if there is no video, but don't make it visible that early if there is video. 5. Set the global fullscreen variable to get aspect scaling. (Hack) 6. In GUI_SETUP_VIDEO_WINDOW: a. First set the video size, b. then resize and reposition, c. then make the window visible, d. switch to/from fullscreen, if necessary. (This minimizes disturbances and the display stays in fullscreen mode and is smoothly black as often as possible.) 7. When playback ends, either act as in 6. or make video window quickly invisible if requested. In actions.c: 8. Allow calling uiFullScreen() even if not playing. 9. Remove unnecessary (and misplaced) repositioning (but not properly resizing) code from uiFullScreen(). 10. Don't call wsFullScreen() conditionally. 11. Only set global variable vo_fs, not the global fullscreen variable.

author	ib
date	Thu, 01 Sep 2011 20:15:40 +0000
parents	ef089a8affc6
children	30a443c59053

comparison

equal deleted inserted replaced

-:ef089a8affc6
+:1cd81338af07
 #ifdef CONFIG_DVDREAD
 #include "stream/stream_dvd.h"
 #endif
 guiInterface_t guiInfo = {
-.VideoWindow = True,
 .StreamType  = STREAMTYPE_DUMMY,
 .Balance     = 50.0f
 };
 static int initialized;
 if (!guiApp.mainDecoration)
 wsWindowDecoration(&guiApp.mainWindow, 0);
 wsVisibleWindow(&guiApp.mainWindow, wsShowWindow);
-#if 0
-wsVisibleWindow(&guiApp.subWindow, wsShowWindow);
-{
-XEvent xev;
-do
-XNextEvent(wsDisplay, &xev);
-while (xev.type != MapNotify || xev.xmap.event != guiApp.subWindow.WindowID);
-guiApp.subWindow.Mapped = wsMapped;
-}
-if (!fullscreen)
-fullscreen = gtkLoadFullscreen;
-if (fullscreen) {
-uiFullScreen();
-btnModify(evFullScreen, btnPressed);
-}
-#else
-if (!fullscreen)
-fullscreen = gtkLoadFullscreen;
 if (gtkShowVideoWindow) {
 wsVisibleWindow(&guiApp.subWindow, wsShowWindow);
 {
 XEvent xev;
 do
 XNextEvent(wsDisplay, &xev);
 while (xev.type != MapNotify || xev.xmap.event != guiApp.subWindow.WindowID);
 guiApp.subWindow.Mapped = wsMapped;
-}
+guiInfo.VideoWindow     = True;
+}
-if (fullscreen) {
+if (gtkLoadFullscreen)
 uiFullScreen();
-btnModify(evFullScreen, btnPressed);
+} else
-}
+wsSetBackgroundRGB(&guiApp.subWindow, 0, 0, 0);
-} else {
-if (fullscreen) {
+if (gtkLoadFullscreen)
-wsVisibleWindow(&guiApp.subWindow, wsShowWindow);
+btnModify(evFullScreen, btnPressed);
-{
-XEvent xev;
-do
-XNextEvent(wsDisplay, &xev);
-while (xev.type != MapNotify || xev.xmap.event != guiApp.subWindow.WindowID);
-guiApp.subWindow.Mapped = wsMapped;
-}
-guiInfo.Playing = GUI_PAUSE; // because of !gtkShowVideoWindow...
-uiFullScreen();          // ...guiInfo.Playing is required
-wsVisibleWindow(&guiApp.subWindow, wsHideWindow);
-btnModify(evFullScreen, btnPressed);
-}
-}
-#endif
 guiInfo.Playing = GUI_STOP;
 uiSubRender = 1;
 mplayer(MPLAYER_SET_EQUALIZER, 0, &eq);
 }
 }
 }
-wsVisibleWindow(&guiApp.subWindow, (guiInfo.VideoWindow ? wsShowWindow : wsHideWindow));
+// These must be done here (in the last call from MPlayer before
+// playback starts) and not in GUI_SETUP_VIDEO_WINDOW, because...
+// ...without video there will be no call to GUI_SETUP_VIDEO_WINDOW
+if (!guiInfo.VideoWindow) {
+wsVisibleWindow(&guiApp.subWindow, wsHideWindow);
+btnModify(evFullScreen, gtkLoadFullscreen ? btnPressed : btnReleased);
+}
+// ...option variable fullscreen determines whether MPlayer will handle
+//    the window given by WinID as fullscreen window (and will do aspect
+//    scaling then) or not - quite rubbish
+fullscreen = gtkLoadFullscreen;
 break;
 case GUI_SET_MIXER:
 if (mixer) {
 float l, r;
 uiEventHandling(evRedraw, 0);
 break;
 case GUI_SETUP_VIDEO_WINDOW:
-if (!guiApp.subWindow.isFullScreen) {
-wsResizeWindow(&guiApp.subWindow, vo_dwidth, vo_dheight);
-wsMoveWindow(&guiApp.subWindow, True, guiApp.sub.x, guiApp.sub.y);
-}
 guiInfo.VideoWidth  = vo_dwidth;
 guiInfo.VideoHeight = vo_dheight;
+if (!guiApp.subWindow.isFullScreen || !guiApp.subWindow.Mapped) {
+if (!guiApp.subWindow.isFullScreen)
+wsResizeWindow(&guiApp.subWindow, guiInfo.VideoWidth, guiInfo.VideoHeight);
+wsMoveWindow(&guiApp.subWindow, True, guiApp.sub.x, guiApp.sub.y);
+if (!guiApp.subWindow.Mapped)
+wsVisibleWindow(&guiApp.subWindow, wsShowWindow);
+}
+if (gtkLoadFullscreen ^ guiApp.subWindow.isFullScreen)
+uiEventHandling(evFullScreen, 0);
 if (guiWinID >= 0)
-wsMoveWindow(&guiApp.mainWindow, False, 0, vo_dheight);
+wsMoveWindow(&guiApp.mainWindow, False, 0, guiInfo.VideoHeight);
 break;
 case GUI_HANDLE_X_EVENT:
 wsEvents(wsDisplay, data);
 guiInfo.Track   = 1;
 guiInfo.Chapter = 1;
 guiInfo.Angle   = 1;
 #endif
-if (!guiApp.subWindow.isFullScreen && gtkShowVideoWindow) {
+if (gtkShowVideoWindow) {
-wsResizeWindow(&guiApp.subWindow, guiApp.sub.width, guiApp.sub.height);
+guiInfo.VideoWindow = True;
-wsMoveWindow(&guiApp.subWindow, True, guiApp.sub.x, guiApp.sub.y);
+guiInfo.VideoWidth  = guiApp.sub.width;
-} else
+guiInfo.VideoHeight = guiApp.sub.height;
+if (!guiApp.subWindow.isFullScreen) {
+wsResizeWindow(&guiApp.subWindow, guiInfo.VideoWidth, guiInfo.VideoHeight);
+wsMoveWindow(&guiApp.subWindow, True, guiApp.sub.x, guiApp.sub.y);
+}
+if (!guiApp.subWindow.Mapped)
+wsVisibleWindow(&guiApp.subWindow, wsShowWindow);
+if (gtkLoadFullscreen ^ guiApp.subWindow.isFullScreen)
+uiEventHandling(evFullScreen, 0);
+} else {
 wsVisibleWindow(&guiApp.subWindow, wsHideWindow);
+guiInfo.VideoWindow = False;
+btnModify(evFullScreen, gtkLoadFullscreen ? btnPressed : btnReleased);
+}
 gui(GUI_SET_STATE, (void *)GUI_STOP);
 wsHandleEvents();
 uiSubRender = 1;

Mercurial > mplayer.hg

comparison gui/interface.c @ 33960:1cd81338af07