<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: dfaure.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19583">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>It is not really meant to be changing after the job started, but there can be instances where the job first has to examine the real destination and only can then set it and start.<br />
This patch ensures that at least when the job has finished, the destUrl is correct so that the job view can provide a more useful finished notification.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>In plasma-browser-integration I only get the destination after the job has been started.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R288 KJobWidgets</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19583">https://phabricator.kde.org/D19583</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kuiserverjobtracker.cpp</div></div></div><br /><div><strong>To: </strong>broulik, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>