- slight simplification of XineControl constructor

- fixed a race between the video resizing and the xine initialization

5 files changed, 37 insertions, 18 deletions

diff --git a/noncore/multimedia/opieplayer2/mediaplayer.cpp b/noncore/multimedia/opieplayer2/mediaplayer.cpp
index 0d9a5b8..c230d6f 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayer.cpp
@@ -365,5 +365,5 @@ void MediaPlayer::recreateAudioAndVideoWidgets()
     connect( videoUI,  SIGNAL( lessReleased() ), this, SLOT( stopChangingVolume() ) );
 
-    xineControl = new XineControl( videoUI, videoUI->vidWidget(), mediaPlayerState );
+    xineControl = new XineControl( videoUI->vidWidget(), mediaPlayerState );
     connect( xineControl, SIGNAL( initialized() ),
              &mediaPlayerState, SLOT( setBackendInitialized() ) );
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.cpp b/noncore/multimedia/opieplayer2/xinecontrol.cpp
index 14e71d9..e791c3b 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.cpp
+++ b/noncore/multimedia/opieplayer2/xinecontrol.cpp
@@ -39,14 +39,14 @@
 #include "xinecontrol.h"
 #include "mediaplayerstate.h"
+#include "xinevideowidget.h"
 
-XineControl::XineControl( QWidget *videoContainerWidget, XineVideoWidget *xineWidget, 
+XineControl::XineControl( XineVideoWidget *xineWidget, 
                           MediaPlayerState &_mediaPlayerState, 
                           QObject *parent, const char *name )
-    : QObject( parent, name ), mediaPlayerState( _mediaPlayerState )
+    : QObject( parent, name ), mediaPlayerState( _mediaPlayerState ), xineVideoWidget( xineWidget )
 {
 
     libXine = new XINE::Lib( xineWidget );
 
-    connect ( videoContainerWidget, SIGNAL( videoResized( const QSize & )), this, SLOT( videoResized ( const QSize & ) ) );
     connect( &mediaPlayerState, SIGNAL( pausedToggled( bool ) ),  this, SLOT( pause( bool ) ) );
     connect( this, SIGNAL( positionChanged( long ) ), &mediaPlayerState, SLOT( updatePosition( long ) ) );
@@ -56,5 +56,5 @@ XineControl::XineControl( QWidget *videoContainerWidget, XineVideoWidget *xineWi
     connect( &mediaPlayerState,  SIGNAL( videoGammaChanged( int ) ), this,  SLOT( setGamma( int ) ) );
     connect( libXine, SIGNAL( stopped() ), this, SLOT( nextMedia() ) );
-    connect( libXine, SIGNAL( initialized() ), this, SIGNAL( initialized() ) );
+    connect( libXine, SIGNAL( initialized() ), this, SLOT( xineInitialized() ) );
 
     disabledSuspendScreenSaver = FALSE;
@@ -127,4 +127,12 @@ void XineControl::setGamma( int value ) {
 }
 
+void XineControl::xineInitialized()
+{
+    connect( xineVideoWidget, SIGNAL( videoResized( const QSize & ) ), this, SLOT( videoResized ( const QSize & ) ) );
+    libXine->resize( xineVideoWidget->videoSize() );
+
+    emit initialized();
+}
+
 void XineControl::stop( bool isSet ) {
     if ( !isSet ) {
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.h b/noncore/multimedia/opieplayer2/xinecontrol.h
index 000529c..085de3f 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.h
+++ b/noncore/multimedia/opieplayer2/xinecontrol.h
@@ -43,5 +43,5 @@ class XineControl : public QObject  {
     Q_OBJECT
 public:
-    XineControl( QWidget *videoContainerWidget, XineVideoWidget *xineWidget, 
+    XineControl( XineVideoWidget *xineWidget, 
                  MediaPlayerState &_mediaPlayerState, 
                  QObject *parent = 0, const char *name =0 );
@@ -102,4 +102,7 @@ public slots:
 
 
+private slots:
+    void xineInitialized();
+
 private:
     XINE::Lib *libXine;
@@ -112,4 +115,5 @@ private:
     bool hasAudioChannel : 1;
     MediaPlayerState &mediaPlayerState;
+    XineVideoWidget *xineVideoWidget;
 
 signals:
diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.cpp b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
index 1d88cea..791818e 100644
--- a/noncore/multimedia/opieplayer2/xinevideowidget.cpp
+++ b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
@@ -109,4 +109,19 @@ void XineVideoWidget::clear ( )
 }
 
+QSize XineVideoWidget::videoSize() const
+{
+    QSize s = size();
+    bool fs = ( s == qApp->desktop()->size() );
+
+    // if we are in fullscreen mode, do not rotate the video
+    // (!! the paint routine uses m_rotation + qt_screen-> transformOrientation() !!)
+    m_rotation = fs ? - qt_screen->transformOrientation() : 0;
+
+    if ( fs && qt_screen->isTransformed() )
+        s = qt_screen->mapToDevice( s );
+
+    return s;
+}
+
 void XineVideoWidget::paintEvent ( QPaintEvent * )
 {
@@ -252,15 +267,5 @@ void XineVideoWidget::setVideoFrame ( uchar* img, int w, int h, int bpl )
 void XineVideoWidget::resizeEvent ( QResizeEvent * )
 {
-        QSize s = size ( );
-        bool fs = ( s == qApp-> desktop ( )-> size ( ));
-
-        // if we are in fullscreen mode, do not rotate the video
-        // (!! the paint routine uses m_rotation + qt_screen-> transformOrientation() !!)
-        m_rotation = fs ? -qt_screen-> transformOrientation ( ) : 0;
-
-        if ( fs && qt_screen-> isTransformed ( ))
-                s = qt_screen-> mapToDevice ( s );
-
-        emit videoResized ( s );
+    emit videoResized( videoSize() );
 }
 
diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.h b/noncore/multimedia/opieplayer2/xinevideowidget.h
index 33f1470..8b3a3ea 100644
--- a/noncore/multimedia/opieplayer2/xinevideowidget.h
+++ b/noncore/multimedia/opieplayer2/xinevideowidget.h
@@ -51,4 +51,6 @@ public:
     void clear ( );
 
+    QSize videoSize() const;
+
 protected:
     void paintEvent( QPaintEvent *p );
@@ -70,5 +72,5 @@ private:
     int     m_bytes_per_pixel;
     QImage *m_logo;
-    int     m_rotation;
+    mutable int     m_rotation;
 };