homescreenhandler: use applaunchd for app activation 47/26947/5 12.91.0 marlin/12.91.0 marlin_12.91.0
authorArnaud Ferraris <arnaud.ferraris@collabora.com>
Wed, 1 Dec 2021 15:51:52 +0000 (16:51 +0100)
committerArnaud Ferraris <arnaud.ferraris@collabora.com>
Mon, 20 Dec 2021 20:04:12 +0000 (21:04 +0100)
As part of the App FW rework, we introduced a new app launcher service,
accessible through D-Bus. This service accepts requests to activate
applications, and notifies when a application successfully activated or
terminated.

Requests for activation are handled by a separate `launcher` application
for now, so we only need to listen on incoming signals in order to react
to those:
- when an application successfully started or is switched to, ask the
  compositor to activate it
- when an application terminates, activate the launcher so the user
  isn't facing a "blank" display

Bug-AGL: SPEC-4160

Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
Change-Id: I7369944570651b45ec9dcca9ccde3dfd75719c3b

homescreen/homescreen.pro
homescreen/src/homescreenhandler.cpp
homescreen/src/homescreenhandler.h

index 0bf98fb..7e93e02 100644 (file)
 
 TEMPLATE = app
 TARGET = homescreen
-QT = qml quick gui-private
+QT = qml quick gui-private dbus
 CONFIG += c++11 link_pkgconfig wayland-scanner
 PKGCONFIG += wayland-client
 
+DBUS_INTERFACES = $$[QT_SYSROOT]/usr/share/dbus-1/interfaces/org.automotivelinux.AppLaunch.xml
+
 SOURCES += \
     src/main.cpp \
     src/shell.cpp \
@@ -37,7 +39,6 @@ HEADERS  += \
     src/mastervolume.h \
     src/homescreenhandler.h
 
-
 OTHER_FILES += \
     README.md
 
index 836ed47..ee8d614 100644 (file)
@@ -22,6 +22,9 @@
 
 #include <qpa/qplatformnativeinterface.h>
 
+#define APPLAUNCH_DBUS_IFACE     "org.automotivelinux.AppLaunch"
+#define APPLAUNCH_DBUS_OBJECT    "/org/automotivelinux/AppLaunch"
+
 void* HomescreenHandler::myThis = 0;
 
 HomescreenHandler::HomescreenHandler(Shell *_aglShell, ApplicationLauncher *launcher, QObject *parent) :
@@ -29,6 +32,8 @@ HomescreenHandler::HomescreenHandler(Shell *_aglShell, ApplicationLauncher *laun
     aglShell(_aglShell)
 {
     mp_launcher = launcher;
+    applaunch_iface = new org::automotivelinux::AppLaunch(APPLAUNCH_DBUS_IFACE, APPLAUNCH_DBUS_OBJECT,
+                                                          QDBusConnection::sessionBus(), this);
 }
 
 HomescreenHandler::~HomescreenHandler()
@@ -48,6 +53,14 @@ void HomescreenHandler::init(void)
 #endif
     myThis = this;
 
+    /*
+     * The "started" signal is received any time a start request is made to applaunchd,
+     * and the application either starts successfully or is already running. This
+     * effectively acts as a "switch to app X" action.
+     */
+    connect(applaunch_iface, SIGNAL(started(QString)), this, SLOT(appStarted(QString)));
+    connect(applaunch_iface, SIGNAL(terminated(QString)), this, SLOT(appTerminated(QString)));
+
 #if 0
     mp_hs->registerCallback(nullptr, HomescreenHandler::onRep_static);
 
@@ -122,6 +135,8 @@ void HomescreenHandler::tapShortcut(QString application_id)
                mp_launcher->setCurrent(application_id);
        }
 #endif
+
+    appStarted(application_id);
 }
 
 #if 0
@@ -155,3 +170,19 @@ void HomescreenHandler::onEv(const string& event, struct json_object* event_cont
     }
 }
 #endif
+
+void HomescreenHandler::appStarted(const QString& application_id)
+{
+    struct agl_shell *agl_shell = aglShell->shell.get();
+    QPlatformNativeInterface *native = qApp->platformNativeInterface();
+    struct wl_output *output = getWlOutput(native, qApp->screens().first());
+
+    HMI_DEBUG("HomeScreen", "Activating application %s", application_id.toStdString().c_str());
+    agl_shell_activate_app(agl_shell, application_id.toStdString().c_str(), output);
+}
+
+void HomescreenHandler::appTerminated(const QString& application_id)
+{
+    HMI_DEBUG("HomeScreen", "Application %s terminated, activating launcher", application_id.toStdString().c_str());
+    appStarted("launcher");
+}
index 6dbf31b..d94740b 100644 (file)
@@ -20,6 +20,7 @@
 #include <QObject>
 
 #include "applicationlauncher.h"
+#include "applaunch_interface.h"
 
 #include "shell.h"
 #include <string>
@@ -50,9 +51,15 @@ public:
 signals:
     void showNotification(QString application_id, QString icon_path, QString text);
     void showInformation(QString info);
+
+public slots:
+    void appStarted(const QString& application_id);
+    void appTerminated(const QString& application_id);
+
 private:
     ApplicationLauncher *mp_launcher;
     Shell *aglShell;
+    org::automotivelinux::AppLaunch *applaunch_iface;
 };
 
 #endif // HOMESCREENHANDLER_H