Dolphin::dolphinGuiInstances should skip daemon process

This function should return a list without
daemonized process, because it is a list of
GUI instances.

So, check whether the found DBus service can
actually call requests for the path `/dolphin/Dolphin_1`,
by calling `isActiveWindow()` method from this interface.

1 files changed, 5 insertions, 1 deletions

diff --git a/src/global.cpp b/src/global.cpp
index c91046efb..b32549102 100644
--- a/src/global.cpp
+++ b/src/global.cpp
@@ -162,7 +162,11 @@ QVector<QPair<QSharedPointer<OrgKdeDolphinMainWindowInterface>, QStringList>> Do
             QSharedPointer<OrgKdeDolphinMainWindowInterface> interface(
                 new OrgKdeDolphinMainWindowInterface(service, QStringLiteral("/dolphin/Dolphin_1"), QDBusConnection::sessionBus()));
             if (interface->isValid() && !interface->lastError().isValid()) {
-                dolphinInterfaces.append(qMakePair(interface, QStringList()));
+                auto isActiveWindowReply = interface->isActiveWindow();
+                isActiveWindowReply.waitForFinished();
+                if (!isActiveWindowReply.isError()) {
+                    dolphinInterfaces.append(qMakePair(interface, QStringList()));
+                }
             }
         }
     }