Fix GDI handle leak when clearing MenuItem images

When setting a null image on a MenuItem, the existing bitmap handle was overwritten with 0 without being released, causing a GDI handle leak if the MenuItem previously had an icon. This commit makes sure the handle is always deleted if it was set before.

Author: arunjose696

bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/MenuItem.java

@@ -807,6 +807,7 @@ private void updateImage () {
 		info.hbmpItem = OS.HBMMENU_CALLBACK;
 	} else {
 		if (OS.IsAppThemed ()) {
+			if (hBitmap != 0) OS.DeleteObject (hBitmap);
 			hBitmap = getMenuItemIconBitmapHandle(image);
 			if ((style & (SWT.CHECK | SWT.RADIO)) != 0 && CUSTOM_SELECTION_IMAGE > 0) {
 				info.fMask |= OS.MIIM_CHECKMARKS;
@@ -895,7 +896,6 @@ private long getMenuItemIconBitmapHandle(Image image) {
 	if (image == null) {
 		return 0;
 	}
-	if (hBitmap != 0) OS.DeleteObject (hBitmap);
 	int zoom = adaptZoomForMenuItem(nativeZoom, image);
 	return Display.create32bitDIB (image, zoom);
 }