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.