User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111114 Firefox/11.0a1 Build ID: 20111114031820 Steps to reproduce: http://hg.mozilla.org/mozilla-central/file/9ae1d4f44b8b/image/src/imgLoader.cpp#l108 While playing around with writing a toy image decoder (in js) this code kept showing up in searches and confusing me. It's been horribly broken since bug 513681 and it's been there since pre-hg.
Created attachment 654600 [details] [diff] [review] Remove confusing dead code implying that image decoders are pluggable. This is trivial really, but I did build and run it on my machine anyway. I used getReviewer.py to find a reviewer, so if it's not you apologies but I'm just following the instructions :P
> I used getReviewer.py to find a reviewer, so if it's not you apologies but I'm just > following the instructions :P I'm a fine reviewer for this code. But also, choosing a poor reviewer is not a faux pas; the reviewer should redirect your request towards someone else without complaint. :) Thanks for the patch!
Normally we'd ask for a patch formatted for checkin-needed  before pushing, but since this is your first one, I've added that info and pushed: https://hg.mozilla.org/integration/mozilla-inbound/rev/16e1ff49a6a5 Thanks again for the patch!  http://blog.bonardo.net/2010/06/22/so-youre-about-to-use-checkin-needed