Remove ability to instantiate a RasterImage by contract ID

NEW
Unassigned

Status

()

Core
ImageLib
7 years ago
4 years ago

People

(Reporter: dholbert, Unassigned)

Tracking

Trunk
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
mozilla-central only instantiates a RasterImage by contract ID in one place -- in imgRequest.cpp, which could just as easily do "new RasterImage()".

Is there any reason we need to even allow creation of RasterImage by contract ID? If not, it seems like extra code that we can & should remove.
(In reply to comment #0)
> mozilla-central only instantiates a RasterImage by contract ID in one place --
> in imgRequest.cpp, which could just as easily do "new RasterImage()".
> 
> Is there any reason we need to even allow creation of RasterImage by contract
> ID?

Nope - we should get rid of it. It should probably wait until I land my big patch stack at bug 513681, since that changes a lot of the nsImageModule stuff.
You need to log in before you can comment on or make changes to this bug.