D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-31 Thread Martin Flöser
graesslin abandoned this revision. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8163 To: graesslin, #kwin, #plasma, mart Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart

D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-17 Thread Marco Martin
mart added a comment. deadlock fixed in sni, https://phabricator.kde.org/D8166 should replace this REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8163 To: graesslin, #kwin, #plasma, mart Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, al

D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-06 Thread Martin Flöser
graesslin added a comment. Just the DBus protocol is in https://phabricator.kde.org/D8166 REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8163 To: graesslin, #kwin, #plasma, mart Cc: davidedmundson, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed

D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-06 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D8163#152777, @davidedmundson wrote: > Immediately going for a workaround is.. (and I don't want to use this term as you've clearly made a huge patch) lazy. Hey, I discussed this with Marco yesterday afternoon ;-) If y

D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-06 Thread David Edmundson
davidedmundson added a comment. Immediately going for a workaround is.. (and I don't want to use this term as you've clearly made a huge patch) lazy. Especially as the current code path in your trace shows we're doing something very stupid: - we go through KStatusNotifierItem - we

D8163: Move StatusNotifierItem for VirtualKeyboard into a helper process

2017-10-06 Thread Martin Flöser
graesslin created this revision. graesslin added reviewers: KWin, Plasma, mart. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY During startup the StatusNotifierItem can cause a freeze as it might do a blocking DBus c