Closed Bug 211066 Opened 21 years ago Closed 14 years ago

Move from deprecated LoadIcon() to LoadImage()

Categories

(Core :: XUL, enhancement, P4)

Product:
Core
Component:
XUL
Platform:
x86
Windows XP
Type:
enhancement

Tracking

()

Status:
RESOLVED WORKSFORME
Milestone:
Future

People

(Reporter: d_king, Unassigned)

References

(
URL
)

Details

As per the above URL from a Microsoft MSDN article, LoadIcon (and LoadIconW)
have been deprecated and replaced by LoadImage (and LoadImageW).

I have marked this as an enhancement as it isn't the cause of any leaks (GDI or
otherwise) that I've found, and thus is just about keeping up with technology.
Note: LoadImage does require a Destroy process if it isn't loaded with sharing
enabled.
http://lxr.mozilla.org/seamonkey/search?string=LoadIcon%5C%28 shows no calls to
LoadIcon in Mozilla GFX code (or anywhere in the Gecko core for that matteR). 
There are several callers in test apps, the ActiveX plugin, plugin samples,
xpinstall, and mailnews -- the best course of action is probably to file one bug
per module involved, assigned to the right components....
Changing to XP Toolkit/Widgets for code in /widget/src/windows/
Component: GFX: Win32 → XP Toolkit/Widgets
-> XP Toolkit/Widgets owner
Assignee: kmcclusk → jaggernaut
Priority: -- → P4
QA Contact: ian → jrgm
Target Milestone: --- → Future
Assignee: jag → nobody
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.