Diffusion KIO dc601bd406b5

KRun: add runApplication method.

Authored by dfaure on Jun 15 2016, 7:30 PM.

Description

KRun: add runApplication method.

Summary:
The main purpose compared to runService is to use QProcess directly rather
than KLauncher. In e.g. Plasma launchers, using KLauncher can block the
GUI for a long time, for no purpose since we're not actually going to make DBus
calls to it. In the long run, we could move towards DBus activation for such cases,
and runApplication otherwise.

While making a new method, I also took the opportunity to turn the bool tempFiles
to a QFlags.

The purpose of this is to be able to clean up the plasma fix https://phabricator.kde.org/D1898
one day.

Test Plan: Added a testcase to the unittest.

Reviewers: hein, broulik, davidedmundson, svuorela

Reviewed By: davidedmundson, svuorela

Subscribers: Plasma, Frameworks

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

Details

Committed
dfaureJun 16 2016, 8:42 PM
Reviewer
davidedmundson
Differential Revision
D1902: KRun: add runApplication method.
Parents
R241:47983cc093f9: Replace obsolete QStyleOption subclasses
Branches
Unknown
Tags
Unknown