[frameworks/kwallet] src/runtime/ksecretd: Drop dead screensaver integration

Nicolas Fella <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit c67e7dcb37fa0a51aa2d5ce3fd5a02c93628f558 by Nicolas Fella.
Committed on 24/07/2026 at 12:32.
Pushed by nicolasfella into branch 'master'.

Drop dead screensaver integration

Q_WS_X11 isn't defined, so this code hasn't been built for years

M  +0    -1    src/runtime/ksecretd/autotests/mockkwalletd.cpp
M  +0    -45   src/runtime/ksecretd/ksecretd.cpp
M  +0    -9    src/runtime/ksecretd/ksecretd.h

https://invent.kde.org/frameworks/kwallet/-/commit/c67e7dcb37fa0a51aa2d5ce3fd5a02c93628f558

diff --git a/src/runtime/ksecretd/autotests/mockkwalletd.cpp b/src/runtime/ksecretd/autotests/mockkwalletd.cpp
index ed2ebe6f..95062efd 100644
--- a/src/runtime/ksecretd/autotests/mockkwalletd.cpp
+++ b/src/runtime/ksecretd/autotests/mockkwalletd.cpp
@@ -118,7 +118,6 @@ MOCK_FUNCTION(KSecretD, sync, 2, );
 MOCK_FUNCTION(KSecretD, changePassword, 3, );
 MOCK_FUNCTION(KSecretD, reconfigure, 0, );
 MOCK_FUNCTION(KSecretD, closeAllWallets, 0, );
-MOCK_FUNCTION(KSecretD, screenSaverChanged, 1, );
 
 void KSecretD::slotServiceOwnerChanged(const QString &, const QString &, const QString &)
 {
diff --git a/src/runtime/ksecretd/ksecretd.cpp b/src/runtime/ksecretd/ksecretd.cpp
index a3e74da4..7ec15155 100644
--- a/src/runtime/ksecretd/ksecretd.cpp
+++ b/src/runtime/ksecretd/ksecretd.cpp
@@ -145,10 +145,6 @@ KSecretD::KSecretD()
         new KWalletPortalSecrets(this);
     }
 
-#ifdef Q_WS_X11
-    screensaver = 0;
-#endif
-
     reconfigure();
     //  KGlobal::dirs()->addResourceType("kwallet", 0, "share/apps/kwallet");
     _dw = new KDirWatch(this);
@@ -168,10 +164,6 @@ KSecretD::KSecretD()
 
 KSecretD::~KSecretD()
 {
-#ifdef Q_WS_X11
-    delete screensaver;
-    screensaver = 0;
-#endif
     closeAllWallets();
     qDeleteAll(_transactions);
 }
@@ -186,21 +178,6 @@ QString KSecretD::decodeWalletName(const QString &mangledName)
     return KWallet::Backend::decodeWalletName(mangledName);
 }
 
-#ifdef Q_WS_X11
-void KSecretD::connectToScreenSaver()
-{
-    screensaver = new QDBusInterface("org.freedesktop.ScreenSaver", "/ScreenSaver", "org.freedesktop.ScreenSaver");
-    if (!screensaver->isValid()) {
-        qCDebug(KSECRETD_LOG) << "Service org.freedesktop.ScreenSaver not found. Retrying in 10 seconds...";
-        // keep attempting every 10 seconds
-        QTimer::singleShot(10000, this, SLOT(connectToScreenSaver()));
-    } else {
-        connect(screensaver, SIGNAL(ActiveChanged(bool)), SLOT(screenSaverChanged(bool)));
-        qCDebug(KSECRETD_LOG) << "connected to screen saver service.";
-    }
-}
-#endif
-
 int KSecretD::generateHandle()
 {
     int rc;
@@ -1693,21 +1670,6 @@ void KSecretD::reconfigure()
     int timeSave = _idleTime;
     // in minutes!
     _idleTime = walletGroup.readEntry("Idle Timeout", 10) * 60 * 1000;
-#ifdef Q_WS_X11
-    if (walletGroup.readEntry("Close on Screensaver", false)) {
-        // BUG 254273 : if KSecretD starts before the screen saver, then the
-        // connection fails and KSecretD never receives it's notifications
-        // To fix this, we use a timer and perform periodic connection
-        // attempts until connection succeeds
-        QTimer::singleShot(0, this, SLOT(connectToScreenSaver()));
-    } else {
-        if (screensaver && screensaver->isValid()) {
-            screensaver->disconnect(SIGNAL(ActiveChanged(bool)), this, SLOT(screenSaverChanged(bool)));
-            delete screensaver;
-            screensaver = 0;
-        }
-    }
-#endif
     // Handle idle changes
     if (_closeIdle) {
         if (_idleTime != timeSave) { // Timer length changed
@@ -1846,13 +1808,6 @@ QString KSecretD::localWallet()
     return KWallet::Wallet::LocalWallet();
 }
 
-void KSecretD::screenSaverChanged(bool s)
-{
-    if (s) {
-        closeAllWallets();
-    }
-}
-
 void KSecretD::activatePasswordDialog()
 {
     checkActiveDialog();
diff --git a/src/runtime/ksecretd/ksecretd.h b/src/runtime/ksecretd/ksecretd.h
index d250af2f..7c10bbbb 100644
--- a/src/runtime/ksecretd/ksecretd.h
+++ b/src/runtime/ksecretd/ksecretd.h
@@ -164,8 +164,6 @@ public Q_SLOTS:
 
     QString localWallet();
 
-    void screenSaverChanged(bool);
-
     // Open a wallet using a pre-hashed password. This is only useful in cooperation
     // with the kwallet PAM module. It's also less secure than manually entering the
     // password as the password hash is transmitted using D-Bus.
@@ -201,9 +199,6 @@ private Q_SLOTS:
     void notifyFailures();
     void processTransactions();
     void activatePasswordDialog();
-#ifdef Q_WS_X11
-    void connectToScreenSaver();
-#endif
 
 private:
     // Internal - open a wallet
@@ -255,10 +250,6 @@ private:
     QList<KWalletTransaction *> _transactions;
     QPointer<QWidget> activeDialog;
 
-#ifdef Q_WS_X11
-    QDBusInterface *screensaver;
-#endif
-
     // sessions
     KWalletSessionStore _sessions;
     QDBusServiceWatcher _serviceWatcher;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.