From aae24bb07d60b603ee372b0372379379be68dadc Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Fri, 25 Nov 2016 10:15:10 +0000 Subject: [PATCH] Don't register a new service for screenshots Summary: If you're going to have kwin in the service name anyway there's absolutely no benefit to having clients not use the existing org.kde.KWin. Test Plan: Object is still registered. Reviewers: #plasma, graesslin Reviewed By: #plasma, graesslin Subscribers: graesslin, plasma-devel, kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D3357 --- effects/screenshot/screenshot.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/effects/screenshot/screenshot.cpp b/effects/screenshot/screenshot.cpp index e948928318..29dffe45c4 100644 --- a/effects/screenshot/screenshot.cpp +++ b/effects/screenshot/screenshot.cpp @@ -62,13 +62,11 @@ ScreenShotEffect::ScreenShotEffect() { connect ( effects, SIGNAL(windowClosed(KWin::EffectWindow*)), SLOT(windowClosed(KWin::EffectWindow*)) ); QDBusConnection::sessionBus().registerObject(QStringLiteral("/Screenshot"), this, QDBusConnection::ExportScriptableContents); - QDBusConnection::sessionBus().registerService(QStringLiteral("org.kde.kwin.Screenshot")); } ScreenShotEffect::~ScreenShotEffect() { QDBusConnection::sessionBus().unregisterObject(QStringLiteral("/Screenshot")); - QDBusConnection::sessionBus().unregisterService(QStringLiteral("org.kde.kwin.Screenshot")); } #ifdef KWIN_HAVE_XRENDER_COMPOSITING