dfaure updated this revision to Diff 81830. dfaure marked an inline comment as done. dfaure added a comment.
Multiple fixes thanks to review comments; didn't remove the fast path yet (a unittest needs fixup) REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29385?vs=81814&id=81830 BRANCH 2020_05_openurljob REVISION DETAIL https://phabricator.kde.org/D29385 AFFECTED FILES autotests/CMakeLists.txt autotests/krununittest.cpp autotests/openurljobtest.cpp autotests/openurljobtest.h src/core/kurlauthorized.h src/gui/CMakeLists.txt src/gui/applicationlauncherjob.cpp src/gui/applicationlauncherjob.h src/gui/openurljob.cpp src/gui/openurljob.h src/gui/openurljobhandlerinterface.cpp src/gui/openurljobhandlerinterface.h src/widgets/CMakeLists.txt src/widgets/jobuidelegate.cpp src/widgets/kdesktopfileactions.cpp src/widgets/kopenwithdialog.cpp src/widgets/kopenwithdialog.h src/widgets/krun.cpp src/widgets/widgetsopenurljobhandler.cpp src/widgets/widgetsopenurljobhandler.h To: dfaure, ahmadsamir, broulik, meven, kossebau, davidedmundson, nicolasfella, svuorela Cc: feverfew, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
