[kwin] [Bug 385371] DBus times out for StatusNotifierItem

2017-10-17 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=385371

David Edmundson  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
  Latest Commit||https://commits.kde.org/kno
   ||tifications/800908eaec7a0e9
   ||3d7b25b51be0065332eed0427

--- Comment #2 from David Edmundson  ---
Git commit 800908eaec7a0e93d7b25b51be0065332eed0427 by David Edmundson.
Committed on 17/10/2017 at 07:08.
Pushed by davidedmundson into branch 'master'.

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

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.

Test Plan:
killed kded5
created an SNI. It failed into the else branch

Reviewers: #plasma

Subscribers: plasma-devel, #frameworks

Tags: #plasma, #frameworks

Differential Revision: https://phabricator.kde.org/D8321

M  +3-4src/kstatusnotifieritem.cpp

https://commits.kde.org/knotifications/800908eaec7a0e93d7b25b51be0065332eed0427

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 385371] DBus times out for StatusNotifierItem

2017-10-06 Thread Martin Flöser
https://bugs.kde.org/show_bug.cgi?id=385371

Martin Flöser  changed:

   What|Removed |Added

  Flags||ReviewRequest+
URL||https://phabricator.kde.org
   ||/D8163

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 385371] DBus times out for StatusNotifierItem

2017-10-04 Thread Martin Flöser
https://bugs.kde.org/show_bug.cgi?id=385371

Martin Flöser  changed:

   What|Removed |Added

 CC||notm...@gmail.com

--- Comment #1 from Martin Flöser  ---
@Marco: do you have any ideas what we can do here? I think the problem is that
KWin tries to talk to status notifier watcher in blocking DBus while status
notifier watcher (kded?) talks to Wayland.

My first idea was to move the KSni in KWin into a QThread. But after reading
code I don't think that's possible at all...

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 385371] DBus times out for StatusNotifierItem

2017-10-04 Thread Martin Flöser
https://bugs.kde.org/show_bug.cgi?id=385371

Martin Flöser  changed:

   What|Removed |Added

  Flags||Wayland+, X11-

-- 
You are receiving this mail because:
You are watching all bug changes.