[location] Prevent location state changes during emergency calls. JB#57710

AML is ran with the emergency calls and it requires to keep location
services, i.e., GPS on during the call. It must not be allowed for the
user to try to change this during the call.

This change tracks the call state with QMceCallState funcionality. When
state is "Active" and type is "Emergency" all changes made to the
location or GPS status are prevented and warning is logged.

Author: LaakkonenJussi

rpm/nemo-qml-plugin-systemsettings.spec

@@ -22,6 +22,7 @@ BuildRequires:  pkgconfig(Qt5XmlPatterns)
 BuildRequires:  pkgconfig(timed-qt5)
 BuildRequires:  pkgconfig(profile)
 BuildRequires:  pkgconfig(mce) >= 1.21.0
+BuildRequires:  pkgconfig(mce-qt5) >= 1.4.2
 BuildRequires:  pkgconfig(mlite5) >= 0.3.6
 BuildRequires:  pkgconfig(usb-moded-qt5)
 BuildRequires:  pkgconfig(blkid)

src/locationsettings.cpp

@@ -206,6 +206,9 @@ LocationSettingsPrivate::LocationSettingsPrivate(LocationSettings::Mode mode, Lo
                                 "net.connman",
                                 "/net/connman/technology/gps",
                                 "net.connman.Technology"))
+    , m_mceCallState(new QMceCallState(this))
+    , m_callstate(QMceCallState::None)
+    , m_calltype(QMceCallState::Normal)
 {
     loadProviders();
 
@@ -233,6 +236,16 @@ LocationSettingsPrivate::LocationSettingsPrivate(LocationSettings::Mode mode, Lo
                 this, &LocationSettingsPrivate::findGpsTech);
         findGpsTech();
     }
+
+    connect(m_mceCallState, &QMceCallState::validChanged,
+            this, &LocationSettingsPrivate::onCallStateChanged);
+    connect(m_mceCallState, &QMceCallState::stateChanged,
+            this, &LocationSettingsPrivate::onCallStateChanged);
+    connect(m_mceCallState, &QMceCallState::typeChanged,
+            this, &LocationSettingsPrivate::onCallStateChanged);
+
+    /* To check the current status of the call if it is valid */
+    onCallStateChanged();
 }
 
 LocationSettingsPrivate::~LocationSettingsPrivate()
@@ -466,6 +479,12 @@ bool LocationSettings::locationEnabled() const
 void LocationSettings::setLocationEnabled(bool enabled)
 {
     Q_D(LocationSettings);
+
+    if (d->isEmergencyCallActive()) {
+        qWarning() << "Cannot allow to change location enabled status when emergency call is active";
+        return;
+    }
+
     if (enabled != d->m_locationEnabled) {
         d->m_locationEnabled = enabled;
         d->writeSettings();
@@ -482,6 +501,12 @@ bool LocationSettings::gpsEnabled() const
 void LocationSettings::setGpsEnabled(bool enabled)
 {
     Q_D(LocationSettings);
+
+    if (d->isEmergencyCallActive()) {
+        qWarning() << "Cannot allow to change GPS status when emergency call is active";
+        return;
+    }
+
     if (enabled != d->m_gpsEnabled) {
         d->m_gpsEnabled = enabled;
         d->writeSettings();
@@ -814,3 +839,17 @@ void LocationSettingsPrivate::writeSettings()
         }
     }
 }
+
+bool LocationSettingsPrivate::isEmergencyCallActive()
+{
+    return m_callstate == QMceCallState::State::Active && m_calltype == QMceCallState::Type::Emergency;
+}
+
+void LocationSettingsPrivate::onCallStateChanged()
+{
+    if (!m_mceCallState->valid())
+        return;
+
+    m_callstate = m_mceCallState->state();
+    m_calltype = m_mceCallState->type();
+}

src/locationsettings_p.h

@@ -41,6 +41,7 @@
 #include <QHash>
 
 #include <nemo-dbus/interface.h>
+#include <mce-qt5/qmcecallstate.h>
 
 #include <sailfishkeyprovider_processmutex.h>
 
@@ -67,6 +68,7 @@ class LocationSettingsPrivate : public QObject
     void updateOnlineAgpsState(const QString &name, LocationSettings::OnlineAGpsState state);
     LocationSettings::LocationMode calculateLocationMode() const;
     void writeSettings();
+    bool isEmergencyCallActive();
 
     QFileSystemWatcher m_watcher;
     bool m_locationEnabled;
@@ -79,11 +81,15 @@ class LocationSettingsPrivate : public QObject
     NetworkManager *m_connMan;
     NetworkTechnology *m_gpsTech;
     NemoDBus::Interface *m_gpsTechInterface;
+    QMceCallState *m_mceCallState;
+    QMceCallState::State m_callstate;
+    QMceCallState::Type m_calltype;
 
 private slots:
     void readSettings();
     void findGpsTech();
     void gpsTechPropertyChanged(const QString &propertyName, const QDBusVariant &value);
+    void onCallStateChanged();
 };
 
 // TODO: replace this with DBus calls to a central settings service...