D23566: Move code to send signals and set priority of processes to a dedicated object

[Arjen Hiemstra]

ahiemstra added inline comments.

INLINE COMMENTS

> davidedmundson wrote in process_controller.h:89
> Eww, it would be better if we could make this a qwindow, but I see you're limited by current KAuth

Agreed. I did look at what would be needed to do that, but the entire widget thing is embedded pretty deeply in KAuth. Luckily, for QML we can probably just leave this out. We won't get window modality, but that's the only drawback. I added this mostly so we don't have any regressions in the current UI.

> davidedmundson wrote in ksysguardprocesslist.cpp:1316
> If we're invoking this from QML any call that spawns an event loop has the potential to crash.
> 
> box = new KMessageBox()
> box->setAttribute(DeleteOnClose);
> box->show();
> 
> would probably help

That's why the ProcessController does not have this code - my intention was to have the UI bits handled completely by QML, so you end up creating an instance of ProcessController, then just call sendSignal or whatever and get a result back, then QML can handle that result. This code is the ProcessList widget that we shouldn't be using in QML anyway.

REPOSITORY
  R111 KSysguard Library

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

To: ahiemstra, #plasma
Cc: davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190829/effc1c49/attachment.html>