Remove the static instance of a compositor.

All classes that needs to access the compositor instance needs to keep
a reference to it

Change-Id: Ia0d4b04b460200ad3c7907b94c032afe90c18771

5 files changed, 34 insertions, 13 deletions

diff --git a/src/compositor/compositor_api/qwaylandclient.cpp b/src/compositor/compositor_api/qwaylandclient.cpp
index 870d566ee..1b3db5858 100644
--- a/src/compositor/compositor_api/qwaylandclient.cpp
+++ b/src/compositor/compositor_api/qwaylandclient.cpp
@@ -48,8 +48,9 @@ QT_BEGIN_NAMESPACE
 class QWaylandClientPrivate : public QObjectPrivate
 {
 public:
-    QWaylandClientPrivate(wl_client *_client)
-        : client(_client)
+    QWaylandClientPrivate(QWaylandCompositor *compositor, wl_client *_client)
+        : compositor(compositor)
+        , client(_client)
     {
         // Save client credentials
         wl_client_get_credentials(client, &pid, &uid, &gid);
@@ -65,10 +66,10 @@ public:
 
         QWaylandClient *client = reinterpret_cast<Listener *>(listener)->parent;
         Q_ASSERT(client != 0);
-        QtWayland::Compositor::instance()->m_clients.removeOne(client);
         delete client;
     }
 
+    QWaylandCompositor *compositor;
     wl_client *client;
 
     uid_t uid;
@@ -82,8 +83,8 @@ public:
     Listener listener;
 };
 
-QWaylandClient::QWaylandClient(wl_client *client)
-    : QObject(*new QWaylandClientPrivate(client))
+QWaylandClient::QWaylandClient(QWaylandCompositor *compositor, wl_client *client)
+    : QObject(*new QWaylandClientPrivate(compositor, client))
 {
     Q_D(QWaylandClient);
 
@@ -91,6 +92,8 @@ QWaylandClient::QWaylandClient(wl_client *client)
     d->listener.parent = this;
     d->listener.listener.notify = QWaylandClientPrivate::client_destroy_callback;
     wl_client_add_destroy_listener(client, &d->listener.listener);
+
+    compositor->handle()->m_clients.append(this);
 }
 
 QWaylandClient::~QWaylandClient()
@@ -99,9 +102,11 @@ QWaylandClient::~QWaylandClient()
 
     // Remove listener from signal
     wl_list_remove(&d->listener.listener.link);
+
+    d->compositor->handle()->m_clients.removeOne(this);
 }
 
-QWaylandClient *QWaylandClient::fromWlClient(wl_client *wlClient)
+QWaylandClient *QWaylandClient::fromWlClient(QWaylandCompositor *compositor, wl_client *wlClient)
 {
     if (!wlClient)
         return 0;
@@ -120,8 +125,7 @@ QWaylandClient *QWaylandClient::fromWlClient(wl_client *wlClient)
         // bind several times resulting in multiple QWaylandClient
         // instances for the same wl_client therefore we create it from
         // here on demand
-        client = new QWaylandClient(wlClient);
-        QtWayland::Compositor::instance()->m_clients.append(client);
+        client = new QWaylandClient(compositor, wlClient);
     }
 
     return client;
@@ -164,7 +168,8 @@ void QWaylandClient::kill(int sig)
 
 void QWaylandClient::close()
 {
-    QtWayland::Compositor::instance()->waylandCompositor()->destroyClient(this);
+    Q_D(QWaylandClient);
+    d->compositor->destroyClient(this);
 }
 
 QT_END_NAMESPACE
diff --git a/src/compositor/compositor_api/qwaylandclient.h b/src/compositor/compositor_api/qwaylandclient.h
index d2710d5f5..dde36f79d 100644
--- a/src/compositor/compositor_api/qwaylandclient.h
+++ b/src/compositor/compositor_api/qwaylandclient.h
@@ -48,6 +48,7 @@ struct wl_client;
 QT_BEGIN_NAMESPACE
 
 class QWaylandClientPrivate;
+class QWaylandCompositor;
 
 class Q_COMPOSITOR_EXPORT QWaylandClient : public QObject
 {
@@ -60,7 +61,7 @@ class Q_COMPOSITOR_EXPORT QWaylandClient : public QObject
 public:
     ~QWaylandClient();
 
-    static QWaylandClient *fromWlClient(wl_client *wlClient);
+    static QWaylandClient *fromWlClient(QWaylandCompositor *compositor, wl_client *wlClient);
 
     wl_client *client() const;
 
@@ -75,7 +76,7 @@ public Q_SLOTS:
     void close();
 
 private:
-    explicit QWaylandClient(wl_client *client);
+    explicit QWaylandClient(QWaylandCompositor *compositor, wl_client *client);
 };
 
 QT_END_NAMESPACE
diff --git a/src/compositor/compositor_api/qwaylandkeyboard.cpp b/src/compositor/compositor_api/qwaylandkeyboard.cpp
index e02e53fa3..09c0bfc9c 100644
--- a/src/compositor/compositor_api/qwaylandkeyboard.cpp
+++ b/src/compositor/compositor_api/qwaylandkeyboard.cpp
@@ -53,6 +53,18 @@ QWaylandKeyboard::QWaylandKeyboard(QWaylandInputDevice *seat, QObject *parent)
     connect(&d->m_focusDestroyListener, &QWaylandDestroyListener::fired, this, &QWaylandKeyboard::focusDestroyed);
 }
 
+QWaylandInputDevice *QWaylandKeyboard::inputDevice() const
+{
+    Q_D(const QWaylandKeyboard);
+    return d->m_seat;
+}
+
+QWaylandCompositor *QWaylandKeyboard::compositor() const
+{
+    Q_D(const QWaylandKeyboard);
+    return d->m_seat->compositor();
+}
+
 void QWaylandKeyboard::focusDestroyed(void *data)
 {
     Q_UNUSED(data);
@@ -68,7 +80,7 @@ QWaylandClient *QWaylandKeyboard::focusClient() const
     Q_D(const QWaylandKeyboard);
     if (!d->focusResource())
         return Q_NULLPTR;
-    return QWaylandClient::fromWlClient(d->focusResource()->client());
+    return QWaylandClient::fromWlClient(compositor(), d->focusResource()->client());
 }
 
 void QWaylandKeyboard::sendKeyModifiers(QWaylandClient *client, uint serial)
diff --git a/src/compositor/compositor_api/qwaylandkeyboard.h b/src/compositor/compositor_api/qwaylandkeyboard.h
index 6443a081d..d28b5f6d0 100644
--- a/src/compositor/compositor_api/qwaylandkeyboard.h
+++ b/src/compositor/compositor_api/qwaylandkeyboard.h
@@ -88,6 +88,9 @@ class Q_COMPOSITOR_EXPORT QWaylandKeyboard : public QObject, public QWaylandExte
 public:
     QWaylandKeyboard(QWaylandInputDevice *seat, QObject *parent = 0);
 
+    QWaylandInputDevice *inputDevice() const;
+    QWaylandCompositor *compositor() const;
+
     void setFocus(QWaylandSurface *surface);
     virtual void setKeymap(const QWaylandKeymap &keymap);
 
diff --git a/src/compositor/compositor_api/qwaylandsurface.cpp b/src/compositor/compositor_api/qwaylandsurface.cpp
index caf816353..ef7665437 100644
--- a/src/compositor/compositor_api/qwaylandsurface.cpp
+++ b/src/compositor/compositor_api/qwaylandsurface.cpp
@@ -67,7 +67,7 @@ QWaylandSurfacePrivate::QWaylandSurfacePrivate(wl_client *wlClient, quint32 id,
     , closing(false)
     , refCount(1)
     , client(QWaylandClient::fromWlClient(wlClient))
-    , windowType(QWaylandSurface::None)
+    , client(QWaylandClient::fromWlClient(compositor, wlClient))
 {}