D8321: Only check for legacy system tray icons if we're going to make one

David Edmundson noreply at phabricator.kde.org
Sun Oct 15 21:43:42 UTC 2017


davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  On Plasma is a rather pointless check as our QPT uses SNIs
  so we end up checking something we've already checked,
  only in a blocking way - and we're going to no-op anyway.
  
  BUG: 385371

TEST PLAN
  killed kded5
  created an SNI. It failed into the else branch

REPOSITORY
  R289 KNotifications

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D8321

AFFECTED FILES
  src/kstatusnotifieritem.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171015/96ba622e/attachment.html>


More information about the Kde-frameworks-devel mailing list