ngraham added a comment.
Good catch @dfaure, the test does indeed fail after this is patch deployed. I've submitted a fix for the failing test: D11836 <https://phabricator.kde.org/D11836>, and marked this as a dependency. If and when D11836 <https://phabricator.kde.org/D11836> is accepted, can I land both? REPOSITORY R241 KIO BRANCH open-with-usability (branched from master) REVISION DETAIL https://phabricator.kde.org/D11569 To: ngraham, rkflx, #vdg, #frameworks, #dolphin, dfaure, elvisangelaccio, abetts Cc: mmustac, davidedmundson, broulik, dfaure, elvisangelaccio, anthonyfieroni, abetts, michaelh, ngraham