aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnaud Ferraris <arnaud.ferraris@collabora.com>2021-12-01 16:51:52 +0100
committerArnaud Ferraris <arnaud.ferraris@collabora.com>2021-12-20 21:04:12 +0100
commitfda3079ca8ff4a6ec6a67981ba1cdc56a8c0c3ab (patch)
tree8aa4fafd2f1bb18246ce7d94ce8911e143f9845b
parent2b7ca7a6eeb79e03507abb7a16b0503591d2b064 (diff)
homescreenhandler: use applaunchd for app activationmarlin_12.91.0marlin/12.91.012.91.0
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
-rw-r--r--homescreen/homescreen.pro5
-rw-r--r--homescreen/src/homescreenhandler.cpp31
-rw-r--r--homescreen/src/homescreenhandler.h7
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 <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");
+}
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 <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