In unUrlSafe, we only translate the first instance of a "-" -> "+" and "_" -> "/". These replacements should be made global.
Appropriate section of code:
It looks like this piece of code is still used, not 100% sure though. Basing my judgement that this should be a global replace on the nsUrlClassifierUtils version at http://mxr.mozilla.org/mozilla-central/source/toolkit/components/url-classifier/nsUrlClassifierUtils.cpp#401
Created attachment 554277 [details] [diff] [review]
Hi, here is the fix for this bug
Hello Nikolay, do you require assistance in getting this patch checked-in to the tree?
Yep, I think so. It emerged that it's more complicated process than I thought.
>> Hello Nikolay, do you require assistance in getting this patch checked-in to the tree?
> Yep, I think so. It emerged that it's more complicated process than I thought.
Setting the checkin-needed flag.
Thanks for the patch, Nikolay! And sorry for the horribly long delay :(