Changing the logic to destroy temporary handle

Author: ShahzaibIbrahim

bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/graphics/Image.java

@@ -2079,14 +2079,10 @@ public static Image win32_new(Device device, int type, long handle, int nativeZo
 }
 
 private void destroyHandle(int zoom) {
-	zoomLevelToImageHandle.entrySet().removeIf(entry -> {
-		final Integer zoomKey = entry.getKey();
-		if (zoom == zoomKey) {
-			destroyHandle(entry.getValue().handle);
-			return true;
-		}
-		return false;
-	});
+	ImageHandle imageHandle = zoomLevelToImageHandle.remove(zoom);
+	if (imageHandle != null) {
+	     destroyHandle(imageHandle.handle);
+	}
 }
 
 private abstract class AbstractImageProviderWrapper {
@@ -2123,6 +2119,7 @@ private class ImageFileNameProviderWrapper extends AbstractImageProviderWrapper
 	 * ImageFileNameProvider to provide file names at various Zoom levels
 	 */
 	private final ImageFileNameProvider provider;
+	private boolean isDestroyed;
 	private int width;
 	private int height;
 	private final static int BASE_ZOOM = 100;
@@ -2183,7 +2180,12 @@ ImageFileNameProviderWrapper createCopy(Image image) {
 
 	@Override
 	boolean isDisposed() {
-		return false;
+		return this.isDestroyed;
+	}
+
+	@Override
+	protected void destroy() {
+		this.isDestroyed = true;
 	}
 }
 
@@ -2195,6 +2197,7 @@ private class ImageDataProviderWrapper extends AbstractImageProviderWrapper {
 	private final ImageDataProvider provider;
 	private int width;
 	private int height;
+	private boolean isDestroyed;
 	private final static int BASE_ZOOM = 100;
 
 	ImageDataProviderWrapper(ImageDataProvider provider) {
@@ -2229,7 +2232,12 @@ ImageHandle getImageMetadata(int zoom) {
 
 	@Override
 	boolean isDisposed() {
-		return false;
+		return this.isDestroyed;
+	}
+
+	@Override
+	protected void destroy() {
+		this.isDestroyed = true;
 	}
 
 	@Override