<table><tr><td style="">drosca added a comment.
</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/D13648">View Revision</a></tr></table><br /><div><div><p>There was a reason it was like that, because QtWebEngine did in some version set QWebEngineDownloadItem::path() as percent encoded string. I'm not sure in which version it was fixed (if it was?), but in any case this patch seems wrong. If the path() is no longer percent encoded, QUrl::fromPercentEncoded can be just removed, instead of percent encoding the string and then decoding it again.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R875 Falkon</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13648">https://phabricator.kde.org/D13648</a></div></div><br /><div><strong>To: </strong>anmolgautam, drosca<br /><strong>Cc: </strong>falkon, tuomisto, herrold, anmolgautam, SGOrava, iodelay, spoorun, ptabis, navarromorales, cochise, clivej, ach<br /></div>