Make ImageHandle & ImageData write ops synchronous

This commit makes the ImageHandle creation and ImageData caching process
synchronous to avoid any racing condition and inefficient memory usages.

Author: HeikoKlare

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

@@ -158,9 +158,20 @@ ImageHandle getHandleOrCreate(int zoom, Supplier<ImageHandle> creator) {
 				return imageHandle;
 			}
 
-			imageHandle = creator.get();
-			zoomLevelToImageHandle.put(zoom, imageHandle);
-			return imageHandle;
+			synchronized (zoomLevelToImageHandle) {
+				imageHandle = getImageHandle(zoom);
+				if (imageHandle == null) {
+					imageHandle = creator.get();
+					zoomLevelToImageHandle.put(zoom, imageHandle);
+				}
+				return imageHandle;
+			}
+		}
+
+		<T> T runSynchronized(Supplier<T> supplier) {
+			synchronized (zoomLevelToImageHandle) {
+				return supplier.get();
+			}
 		}
 
 		boolean hasImageHandle(ImageHandle imageHandle) {
@@ -1382,7 +1393,13 @@ public ImageData getImageData (int zoom) {
 		return imageHandle.getImageData();
 	}
 
-	return this.imageProvider.newImageData(zoom);
+	return imageHandleManager.runSynchronized(() -> {
+		ImageHandle obtainedImageHandle = imageHandleManager.getImageHandle(zoom);
+		if (obtainedImageHandle != null) {
+			return obtainedImageHandle.getImageData();
+		}
+		return this.imageProvider.newImageData(zoom);
+	});
 }
 
 /**