Refactor Cursor handle to support per-zoom handles

- Removed the dedicated `handle` field from Cursor; all native handles
are now stored in `zoomLevelToHandle`.
- Updated hashCode, equals, isDisposed, and destroy to work with
zoom-level handles.

Author: ShahzaibIbrahim

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

@@ -49,18 +49,6 @@
  */
 public final class Cursor extends Resource {
 
-	/**
-	 * the handle to the OS cursor resource
-	 * (Warning: This field is platform dependent)
-	 * <p>
-	 * <b>IMPORTANT:</b> This field is <em>not</em> part of the SWT
-	 * public API. It is marked public only so that it can be shared
-	 * within the packages provided by SWT. It is not available on all
-	 * platforms and should never be accessed from application code.
-	 * </p>
-	 *
-	 */
-	private long handle;
 	/**
 	 * Attribute to cache current native zoom level
 	 */
@@ -74,6 +62,8 @@ public final class Cursor extends Resource {
 	private final ImageData mask;
 	private final int hotspotX;
 	private final int hotspotY;
+	private boolean isDestroyed;
+	private Integer style;
 /**
  * Prevents uninitialized instances from being created outside the package.
  */
@@ -84,7 +74,6 @@ public final class Cursor extends Resource {
 	this.mask = null;
 	this.hotspotX = -1;
 	this.hotspotY = -1;
-	this.device.registerResourceWithZoomSupport(this);
 }
 
 /**
@@ -135,8 +124,9 @@ public final class Cursor extends Resource {
  */
 public Cursor(Device device, int style) {
 	this(device);
-	this.handle = setupCursorFromStyle(style);
+	this.style = style;
 	init();
+	this.device.registerResourceWithZoomSupport(this);
 }
 
 private static long setupCursorFromStyle(int style) {
@@ -212,7 +202,6 @@ public Cursor(Device device, ImageData source, ImageData mask, int hotspotX, int
 	this.hotspotX = hotspotX;
 	this.hotspotY = hotspotY;
 	this.imageDataProvider = null;
-	this.handle = setupCursorFromImageData(source, mask, hotspotX, hotspotY);
 	init();
 	this.device.registerResourceWithZoomSupport(this);
 }
@@ -285,7 +274,6 @@ public Cursor(Device device, ImageData source, int hotspotX, int hotspotY) {
 	this.hotspotX = hotspotX;
 	this.hotspotY = hotspotY;
 	this.imageDataProvider = null;
-	this.handle = setupCursorFromImageData(device, source, hotspotX, hotspotY);
 	isIcon = true;
 	init();
 	this.device.registerResourceWithZoomSupport(this);
@@ -402,7 +390,6 @@ public Cursor(Device device, ImageDataProvider imageDataProvider, int hotspotX,
 	this.mask = null;
 	this.hotspotX = hotspotX;
 	this.hotspotY = hotspotY;
-	this.handle = setupCursorFromImageData(device, this.source, hotspotX, hotspotY);
 	isIcon = true;
 	init();
 	this.device.registerResourceWithZoomSupport(this);
@@ -425,24 +412,26 @@ public Cursor(Device device, ImageDataProvider imageDataProvider, int hotspotX,
  */
 public static Long win32_getHandle (Cursor cursor, int zoom) {
 	if (cursor.isDisposed()) {
-		return cursor.handle;
+		return 0L;
 	}
 	if (cursor.zoomLevelToHandle.get(zoom) != null) {
 		return cursor.zoomLevelToHandle.get(zoom);
 	}
 
-	if (cursor.source == null) {
-		cursor.setHandleForZoomLevel(cursor.handle, zoom);
+	if (cursor.style != null) {
+		// we don't need to pass zoom in this case. LoadCursor will always return scaled cursor even though handle value will be same for all zoom levels.
+		long handle = setupCursorFromStyle(cursor.style);
+		cursor.setHandleForZoomLevel(handle, zoom);
 	} else {
 		ImageData source;
 		if (cursor.imageDataProvider != null) {
 			Image tempImage = new Image(cursor.getDevice(), cursor.imageDataProvider);
 			source = tempImage.getImageData(zoom);
 			tempImage.dispose();
+		} else {
+			source = DPIUtil.scaleImageData(cursor.getDevice(), cursor.source, zoom, DEFAULT_ZOOM);
 		}
-		else {
-			source = DPIUtil.scaleImageData(cursor.device, cursor.source, zoom, DEFAULT_ZOOM);
-		}
+
 		if (cursor.isIcon) {
 			long handle = setupCursorFromImageData(cursor.getDevice(), source, Win32DPIUtils.pointToPixel(cursor.hotspotX, zoom), Win32DPIUtils.pointToPixel(cursor.hotspotY, zoom));
 			cursor.setHandleForZoomLevel(handle, zoom);
@@ -456,9 +445,6 @@ public static Long win32_getHandle (Cursor cursor, int zoom) {
 }
 
 private void setHandleForZoomLevel(long handle, Integer zoom) {
-	if (this.handle == 0) {
-		this.handle = handle;	// Set handle for default zoom level
-	}
 	if (zoom != null && !zoomLevelToHandle.containsKey(zoom)) {
 		zoomLevelToHandle.put(zoom, handle);
 	}
@@ -480,14 +466,14 @@ void destroy () {
 //	}
 	device.deregisterResourceWithZoomSupport(this);
 	destroyHandle();
+	this.isDestroyed = true;
 }
 
 private void destroyHandle () {
 	for (Long handle : zoomLevelToHandle.values()) {
 		destroyHandle(handle);
 	}
 	zoomLevelToHandle.clear();
-	handle = 0;
 }
 
 private void destroyHandle(long handle) {
@@ -521,7 +507,7 @@ public boolean equals (Object object) {
 	if (object == this) return true;
 	if (!(object instanceof Cursor)) return false;
 	Cursor cursor = (Cursor) object;
-	return device == cursor.device && handle == cursor.handle;
+	return device == cursor.device && win32_getHandle(this, DEFAULT_ZOOM) == win32_getHandle(cursor, DEFAULT_ZOOM);
 }
 
 /**
@@ -536,7 +522,7 @@ public boolean equals (Object object) {
  */
 @Override
 public int hashCode () {
-	return (int)handle;
+	return win32_getHandle(this, DEFAULT_ZOOM).intValue();
 }
 
 /**
@@ -551,7 +537,7 @@ public int hashCode () {
  */
 @Override
 public boolean isDisposed() {
-	return handle == 0;
+	return isDestroyed;
 }
 
 /**
@@ -563,7 +549,7 @@ public boolean isDisposed() {
 @Override
 public String toString () {
 	if (isDisposed()) return "Cursor {*DISPOSED*}";
-	return "Cursor {" + handle + "}";
+	return "Cursor {" + zoomLevelToHandle + "}";
 }
 
 @Override