From fda3079ca8ff4a6ec6a67981ba1cdc56a8c0c3ab Mon Sep 17 00:00:00 2001 From: Arnaud Ferraris Date: Wed, 1 Dec 2021 16:51:52 +0100 Subject: homescreenhandler: use applaunchd for app activation 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 Change-Id: I7369944570651b45ec9dcca9ccde3dfd75719c3b --- homescreen/homescreen.pro | 5 +++-- homescreen/src/homescreenhandler.cpp | 31 +++++++++++++++++++++++++++++++ homescreen/src/homescreenhandler.h | 7 +++++++ 3 files changed, 41 insertions(+), 2 deletions(-) diff --git a/homescreen/homescreen.pro b/homescreen/homescreen.pro index 0bf98fb..7e93e02 100644 --- a/homescreen/homescreen.pro +++ b/homescreen/homescreen.pro @@ -15,10 +15,12 @@ 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 diff --git a/homescreen/src/homescreenhandler.cpp b/homescreen/src/homescreenhandler.cpp index 836ed47..ee8d614 100644 --- a/homescreen/src/homescreenhandler.cpp +++ b/homescreen/src/homescreenhandler.cpp @@ -22,6 +22,9 @@ #include +#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"); +} diff --git a/homescreen/src/homescreenhandler.h b/homescreen/src/homescreenhandler.h index 6dbf31b..d94740b 100644 --- a/homescreen/src/homescreenhandler.h +++ b/homescreen/src/homescreenhandler.h @@ -20,6 +20,7 @@ #include #include "applicationlauncher.h" +#include "applaunch_interface.h" #include "shell.h" #include @@ -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 -- cgit 1.2.3-korg