Blackberry: Compile with lastest event filter API changes

Change-Id: I4eb7ddf050cd8ef34df4547482f53f7372584221
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Reviewed-by: David Faure <faure@kde.org>

src/plugins/blackberry/bbmediaplayercontrol.cpp

@@ -66,29 +66,6 @@ QT_BEGIN_NAMESPACE
 
 static int idCounter = 0;
 
-static bool s_eventFilterInstalled = 0;
-static QAbstractEventDispatcher::EventFilter s_previousEventFilter = 0;
-static QHash< int, BbMediaPlayerControl* > s_idToPlayerMap;
-
-static bool s_eventFilter(void *message)
-{
-    bps_event_t * const event = static_cast<bps_event_t *>(message);
-
-    if (event &&
-        (bps_event_get_domain(event) == mmrenderer_get_domain() ||
-         bps_event_get_domain(event) == screen_get_domain() )) {
-        const int id = mmrenderer_event_get_userdata(event);
-        BbMediaPlayerControl * const control = s_idToPlayerMap.value(id);
-        if (control)
-            control->bpsEventHandler(event);
-    }
-
-    if (s_previousEventFilter)
-        return s_previousEventFilter(message);
-    else
-        return false;
-}
-
 BbMediaPlayerControl::BbMediaPlayerControl(QObject *parent)
     : QMediaPlayerControl(parent),
       m_connection(0),
@@ -107,12 +84,7 @@ BbMediaPlayerControl::BbMediaPlayerControl(QObject *parent)
       m_stopEventsToIgnore(0),
       m_bufferStatus(0)
 {
-    if (!s_eventFilterInstalled) {
+    QCoreApplication::eventDispatcher()->installNativeEventFilter(this);
-        s_eventFilterInstalled = true;
-        s_previousEventFilter =
-                QCoreApplication::eventDispatcher()->setEventFilter(s_eventFilter);
-    }
-
     openConnection();
 }
 
@@ -121,6 +93,7 @@ BbMediaPlayerControl::~BbMediaPlayerControl()
     stop();
     detach();
     closeConnection();
+    QCoreApplication::eventDispatcher()->removeNativeEventFilter(this);
 }
 
 void BbMediaPlayerControl::openConnection()
@@ -142,7 +115,6 @@ void BbMediaPlayerControl::openConnection()
         return;
     }
 
-    s_idToPlayerMap.insert(m_id, this);
     m_eventMonitor = mmrenderer_request_events(m_contextName.toLatin1(), 0, m_id);
     if (!m_eventMonitor) {
         qBbMediaDebug() << "Unable to request multimedia events";
@@ -152,7 +124,6 @@ void BbMediaPlayerControl::openConnection()
 
 void BbMediaPlayerControl::closeConnection()
 {
-    s_idToPlayerMap.remove(m_id);
     if (m_eventMonitor) {
         mmrenderer_stop_events(m_eventMonitor);
         m_eventMonitor = 0;
@@ -527,14 +498,21 @@ void BbMediaPlayerControl::setVideoControl(BbVideoWindowControl *videoControl)
     m_videoControl = videoControl;
 }
 
-void BbMediaPlayerControl::bpsEventHandler(bps_event_t *event)
+bool BbMediaPlayerControl::nativeEventFilter(const QByteArray &eventType, void *message, long *result)
 {
+    Q_UNUSED(eventType);
+    Q_UNUSED(result);
+
+    bps_event_t * const event = static_cast<bps_event_t *>(message);
+    if (!event ||
+        (bps_event_get_domain(event) != mmrenderer_get_domain() &&
+         bps_event_get_domain(event) != screen_get_domain()))
+        return false;
+
     if (m_videoControl)
         m_videoControl->bpsEventHandler(event);
 
-    if (bps_event_get_domain(event) != mmrenderer_get_domain())
+    if (bps_event_get_domain(event) == mmrenderer_get_domain()) {
-        return;
-
         if (bps_event_get_code(event) == MMRENDERER_STATE_CHANGE) {
             const mmrenderer_state_t newState = mmrenderer_event_get_state(event);
             if (newState == MMR_STOPPED) {
@@ -547,7 +525,7 @@ void BbMediaPlayerControl::bpsEventHandler(bps_event_t *event)
                     --m_stopEventsToIgnore;
                 else
                     stopInternal(IgnoreMmRenderer);
-            return;
+                return false;
             }
         }
 
@@ -556,7 +534,7 @@ void BbMediaPlayerControl::bpsEventHandler(bps_event_t *event)
             // Prevent spurious position change events from overriding our own position, for example
             // when setting the position to 0 in stop().
             if (m_state != QMediaPlayer::PlayingState)
-            return;
+                return false;
 
             const qint64 newPosition = QString::fromLatin1(mmrenderer_event_get_position(event)).toLongLong();
             if (newPosition != 0 && newPosition != m_position) {
@@ -575,6 +553,9 @@ void BbMediaPlayerControl::bpsEventHandler(bps_event_t *event)
                 }
             }
         }
+    }
+
+    return false;
 }
 
 void BbMediaPlayerControl::updateMetaData()

src/plugins/blackberry/bbmediaplayercontrol.h

@@ -43,6 +43,7 @@
 
 #include "bbmetadata.h"
 #include <qmediaplayercontrol.h>
+#include <QtCore/qabstractnativeeventfilter.h>
 #include <QtCore/qpointer.h>
 
 struct bps_event_t;
@@ -54,7 +55,7 @@ QT_BEGIN_NAMESPACE
 
 class BbVideoWindowControl;
 
-class BbMediaPlayerControl : public QMediaPlayerControl
+class BbMediaPlayerControl : public QMediaPlayerControl, public QAbstractNativeEventFilter
 {
     Q_OBJECT
 public:
@@ -97,7 +98,7 @@ public:
     void stop();
 
     void setVideoControl(BbVideoWindowControl *videoControl);
-    void bpsEventHandler(bps_event_t *event);
+    bool nativeEventFilter(const QByteArray &eventType, void *message, long *result) Q_DECL_OVERRIDE;
 
 private Q_SLOTS:
     void continueLoadMedia();