1274881 - Get rid of imgLoader::GetInstance

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[Jonathan Watt [:jwatt]]

imgLoader::GetInstance creates a separate imgLoader instance that is used only by two pieces of code. Either these bits of code should be using imgLoader::Singleton like everyone else, or we should document why they need to be using a separate imgLoader instance.

imgLoader::GetInstance was added in bug 801061. It sounds like there might have been some out of band discussion as to why it was introduced, but since that discussion didn't happen in the bug and nobody's bothered to document the method, who knows what the rational was.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 2

[Jonathan Watt [:jwatt]]

(In reply to Jonathan Watt [:jwatt] from comment #0)
> imgLoader::GetInstance was added in bug 801061.

Along with the GetInstance call in nsContextMenuInfo.cpp.

FWIW the call in OSXNotificationCenter.mm came from https://hg.mozilla.org/mozilla-central/rev/e8f0814e85c5 (bug 852648). Again, no explanation as to why we're not using imgLoader::Singleton.

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: patch

Actually OSXNotificationCenter::ShowAlertWithIconData is implementing the method of the same name on interface nsIAlertsIconData, so those icons could be coming from a website. While we're here we should fix this code to use a/the imgLoader that has had RespectPrivacyNotifications() called on it.

Comment 4

[Pulsebot]

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

Comment 5

[Carsten Book [:Tomcat]]

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