Review Request 121233: Do not drop ASN passed to KRun when executing desktop files

Martin Gräßlin mgraesslin at kde.org
Mon Nov 24 18:53:38 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121233/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: kio


Description
-------

KDesktopFileActions::run did not accept the optional argument
QByteArray asn which meant that any application startup id passed
to KRun got dropped at this point when launching a desktop file.

This change ensures that an ASN is passed along. This way we can
properly pass around the application which launched the process
and the timestamp which triggered the launch (otherwise klauncher
needs a roundtrip to the X-Server to fetch the latest timestamp).


Diffs
-----

  src/widgets/kdesktopfileactions.cpp 0cbeb00f0c0344b5b204f0d18b7c269da6c99e7f 
  src/widgets/kdesktopfileactions.h 7b96ff079c752436aebd0b134b930171ea088f48 
  src/widgets/krun.cpp c623b583716e492622e0d1d1c9cec485c6bcf163 

Diff: https://git.reviewboard.kde.org/r/121233/diff/


Testing
-------


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141124/e308ca73/attachment.html>


More information about the Kde-frameworks-devel mailing list