currently the about: protocol is not case insensetive for what comes after the ":". It could be nice if it was. Currently this works: aBout:cache ABOUT:cache but this doesn't: about:CAche ABOUT:CACHE build 20010718 on win2k
Created attachment 49132 [details] [diff] [review] Converts what comes after about: to lowercase before looking for a module
If all the current ones are lower case, I see no problem with this. Bbaetz? Gerv
I thought I commented on this bug. Hmm. The scheme is case insensitive by definition. url paths aren't, but we're not using those. some ns4 about urls are case sensitive, some aren't; I don't really have an opinion. This would stop an about:mOzIlLa easter egg, though. I don't think anyone was planning on that though :)
i don't have any problems with the patch (r/sr=darin)
r=pchen if you need it
Gagan, don't you own the about: protocol handler... Are you happy with this proposed fix?
yeah seems ok to me.
does this patch affect URLs that have case-sensitive parts? Like this for example: about:cache-entry?client=image&sb=0&key=http://www.edconrad.com/ebay/Smith1/MVC-004S.JPG
reassigning to me. I have completely forgotten about this, as I forgot to cc mysef/assign to me when I created the patch. will checkin shortly
Alexey: no, such urls are not affected. checked in.