Closed Bug 1275266 Opened 4 years ago Closed 4 years ago
Loader::Singleton and img Loader::PBSingleton to something less misleading, and document them
The method names Singleton and PBSingleton on imgLoader are misleading, especially at call sites where we only call Singleton(). (We can and do create multiple imgLoader instances.) The meaning of PBSingleton is also not obvious without digging into the code which isn't a very good way to figure out the meaning of an interface.
Attachment #8755859 - Flags: review?(tnikkel) → review+
You need to log in before you can comment on or make changes to this bug.