1275229 - Stop using do_CreateInstance("@mozilla.org/image/loader;1") to create gecko internal imgLoader instances

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Jonathan Watt [:jwatt]]

imgLoader currently uses do_CreateInstance("@mozilla.org/image/loader;1") to create imgLoader instances because it needs its XPCOM module to be initialized before imgLoader instances are created or used, and do_CreateInstance takes care of that. That's silly though. We can just initialize the module ourselves which is more efficient, less confusing, and clears the way to getting rid of "@mozilla.org/image/loader;1" and "@mozilla.org/image/cache;1".

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jonathan Watt [:jwatt]]

FWIW I built without the module initialization to see what breaks. The first thing to break is the sMemReporter->AddRef() call in the imgLoader constructor, since sMemReporter is created during the module initialization step. That was tricky to figure out though because the errors on Try were really opaque (hence why I think this is worth a comment for anyone doing bug archeology in the future).

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c3686b389285&selectedJob=21300550

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a03bd5fc36c0

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a03bd5fc36c0