It would be nice if the image decoders never called back into other code. The callbacks that we get here make lots of things harder to reason about. For example, RasterImage goes through a bunch of trouble to make sure that the decoder doesn't disappear from underneath it. This will also make it much easier to implement threaded image decoding and will also remove the last part of needed XPCOM from Decoder. We should be able to accomplish this through a combination of recording status and returning instead of calling back.
You need to log in before you can comment on or make changes to this bug.