From 439d5b0e53f564dc75e427743ccce75c6a62f1c6 Mon Sep 17 00:00:00 2001
From: Yoann Lopes <yoann.lopes@digia.com>
Date: Fri, 28 Sep 2012 17:53:11 +0200
Subject: QVideoWidget: fixed setFullScreen(false) on the Xcb platform.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fixed exiting fullscreen when a QVideoWidget is embedded in another QWidget.
Requires change I3616dc0f in qtbase to work.
Because of a bug in some X window managers, the order in which
setWindowFlags() and showNormal() are called has been changed.

Change-Id: I335d47a3a1d9ce517978cad35597d72312150db1
Reviewed-by: Samuel Rødal <samuel.rodal@digia.com>
---
 examples/player/videowidget.cpp        |  2 +-
 src/multimediawidgets/qvideowidget.cpp | 15 +++++++--------
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/examples/player/videowidget.cpp b/examples/player/videowidget.cpp
index 0b2d70903..1dfbf94c0 100644
--- a/examples/player/videowidget.cpp
+++ b/examples/player/videowidget.cpp
@@ -57,7 +57,7 @@ VideoWidget::VideoWidget(QWidget *parent)
 void VideoWidget::keyPressEvent(QKeyEvent *event)
 {
     if (event->key() == Qt::Key_Escape && isFullScreen()) {
-        showNormal();
+        setFullScreen(false);
 
         event->accept();
     } else if (event->key() == Qt::Key_Enter && event->modifiers() & Qt::Key_Alt) {
diff --git a/src/multimediawidgets/qvideowidget.cpp b/src/multimediawidgets/qvideowidget.cpp
index a5670d24c..031193df4 100644
--- a/src/multimediawidgets/qvideowidget.cpp
+++ b/src/multimediawidgets/qvideowidget.cpp
@@ -720,9 +720,9 @@ void QVideoWidget::setFullScreen(bool fullScreen)
 {
     Q_D(QVideoWidget);
 
-    if (fullScreen) {
-        Qt::WindowFlags flags = windowFlags();
+    Qt::WindowFlags flags = windowFlags();
 
+    if (fullScreen) {
         d->nonFullScreenFlags = flags & (Qt::Window | Qt::SubWindow);
         flags |= Qt::Window;
         flags &= ~Qt::SubWindow;
@@ -730,6 +730,10 @@ void QVideoWidget::setFullScreen(bool fullScreen)
 
         showFullScreen();
     } else {
+        flags &= ~(Qt::Window | Qt::SubWindow); //clear the flags...
+        flags |= d->nonFullScreenFlags; //then we reset the flags (window and subwindow)
+        setWindowFlags(flags);
+
         showNormal();
     }
 }
@@ -910,13 +914,8 @@ bool QVideoWidget::event(QEvent *event)
             if (d->currentControl)
                 d->currentControl->setFullScreen(false);
 
-            if (d->wasFullScreen) {
-                flags &= ~(Qt::Window | Qt::SubWindow); //clear the flags...
-                flags |= d->nonFullScreenFlags; //then we reset the flags (window and subwindow)
-                setWindowFlags(flags);
-
+            if (d->wasFullScreen)
                 emit fullScreenChanged(d->wasFullScreen = false);
-            }
         }
     }
     return QWidget::event(event);
-- 
cgit v1.2.3