Non system cursors adaption to DPI scale changes

Creating a map to provide different handle w.r.t the current zoom of the
display. The method win32_getHandle provides the appropriate handle for
the scaled image as per requested by the client.

Author: ShahzaibIbrahim

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

@@ -14,7 +14,10 @@
 package org.eclipse.swt.graphics;
 
 
+import java.util.*;
+
 import org.eclipse.swt.*;
+import org.eclipse.swt.internal.*;
 import org.eclipse.swt.internal.win32.*;
 
 /**
@@ -59,9 +62,18 @@ public final class Cursor extends Resource {
 	 * @noreference This field is not intended to be referenced by clients.
 	 */
 	public long handle;
+	/**
+	 * Attribute to cache current native zoom level
+	 */
+	private static int DEFAULT_ZOOM = 100;
 
-	boolean isIcon;
+	private HashMap<Integer, Long> zoomLevelToHandle = new HashMap<>();
 
+	boolean isIcon;
+	private ImageData source;
+	private ImageData mask;
+	private int hotspotX;
+	private int hotspotY;
 /**
  * Prevents uninitialized instances from being created outside the package.
  */
@@ -184,6 +196,10 @@ public Cursor(Device device, int style) {
  */
 public Cursor(Device device, ImageData source, ImageData mask, int hotspotX, int hotspotY) {
 	super(device);
+	this.source = source;
+	this.mask = mask;
+	this.hotspotX = hotspotX;
+	this.hotspotY = hotspotY;
 	if (source == null) SWT.error(SWT.ERROR_NULL_ARGUMENT);
 	if (mask == null) {
 		if (source.getTransparencyType() != SWT.TRANSPARENCY_MASK) {
@@ -246,6 +262,9 @@ public Cursor(Device device, ImageData source, ImageData mask, int hotspotX, int
  */
 public Cursor(Device device, ImageData source, int hotspotX, int hotspotY) {
 	super(device);
+	this.source = source;
+	this.hotspotX = hotspotX;
+	this.hotspotY = hotspotY;
 	if (source == null) SWT.error(SWT.ERROR_NULL_ARGUMENT);
 	/* Check the hotspots */
 	if (hotspotX >= source.width || hotspotX < 0 ||
@@ -319,6 +338,55 @@ public Cursor(Device device, ImageData source, int hotspotX, int hotspotY) {
 	init();
 }
 
+/**
+ * <b>IMPORTANT:</b> This method is not part of the public
+ * API for Image. 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 called from
+ * application code.
+ *
+ * Updates zoom and refresh the Cursor based on the native zoom level, if required.
+ *
+ * @param cursor the cursor to get the handle of
+ * @param zoom device zoom in % of the monitor on which the image is painted
+ *
+ * @return true if cursor is refreshed
+ *
+ * @noreference This method is not intended to be referenced by clients.
+ */
+public static Long win32_getHandle (Cursor cursor, int zoom) {
+	if (cursor.isDisposed()) {
+		return cursor.handle;
+	}
+	if (cursor.zoomLevelToHandle.get(zoom) != null) {
+		return cursor.zoomLevelToHandle.get(zoom);
+	}
+
+	if (cursor.source == null) {
+		cursor.setHandleForZoomLevel(cursor.handle, zoom);
+	} else {
+		ImageData source = DPIUtil.scaleImageData(cursor.device, cursor.source, zoom, DEFAULT_ZOOM);
+		if (cursor.isIcon) {
+			Cursor newCursor = new Cursor(cursor.device, source, cursor.hotspotX, cursor.hotspotY);
+			cursor.setHandleForZoomLevel(newCursor.handle, zoom);
+		} else {
+			ImageData mask = DPIUtil.scaleImageData(cursor.device, cursor.mask, zoom, DEFAULT_ZOOM);
+			Cursor newCursor = new Cursor(cursor.device, source, mask, cursor.hotspotX, cursor.hotspotY);
+			cursor.setHandleForZoomLevel(newCursor.handle, zoom);
+		}
+	}
+	return cursor.zoomLevelToHandle.get(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);
+	}
+}
+
 @Override
 void destroy () {
 	/*
@@ -333,7 +401,18 @@ void destroy () {
 //	if (OS.GetCursor() == handle) {
 //		OS.SetCursor(OS.LoadCursor(0, OS.IDC_ARROW));
 //	}
+	destroyHandle();
+}
+
+private void destroyHandle () {
+	for (Long handle : zoomLevelToHandle.values()) {
+		destroyHandle(handle);
+	}
+	zoomLevelToHandle.clear();
+	handle = 0;
+}
 
+private void destroyHandle(long handle) {
 	if (isIcon) {
 		OS.DestroyIcon(handle);
 	} else {
@@ -347,7 +426,6 @@ void destroy () {
 		*/
 		OS.DestroyCursor(handle);
 	}
-	handle = 0;
 }
 
 /**

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

@@ -5410,7 +5410,7 @@ LRESULT WM_SETCURSOR (long wParam, long lParam) {
 		if (control == null) return null;
 		Cursor cursor = control.findCursor ();
 		if (cursor != null) {
-			OS.SetCursor (cursor.handle);
+			OS.SetCursor (Cursor.win32_getHandle(cursor, getZoom()));
 			return LRESULT.ONE;
 		}
 	}

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

@@ -2610,7 +2610,7 @@ LRESULT WM_SETCURSOR (long wParam, long lParam) {
 				RECT rect = new RECT ();
 				OS.GetClientRect (handle, rect);
 				if (OS.PtInRect (rect, pt)) {
-					OS.SetCursor (cursor.handle);
+					OS.SetCursor (Cursor.win32_getHandle(cursor, getZoom()));
 					switch (msg) {
 						case OS.WM_LBUTTONDOWN:
 						case OS.WM_RBUTTONDOWN:

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

@@ -823,7 +823,7 @@ public void setCursor(Cursor newCursor) {
 	checkWidget();
 	clientCursor = newCursor;
 	if (newCursor != null) {
-		if (inEvent) OS.SetCursor (clientCursor.handle);
+		if (inEvent) OS.SetCursor (Cursor.win32_getHandle(clientCursor, getZoom()));
 	}
 }
 
@@ -892,7 +892,7 @@ long transparentProc (long hwnd, long msg, long wParam, long lParam) {
 			break;
 		case OS.WM_SETCURSOR:
 			if (clientCursor != null) {
-				OS.SetCursor (clientCursor.handle);
+				OS.SetCursor (Cursor.win32_getHandle(clientCursor, getZoom()));
 				return 1;
 			}
 			if (resizeCursor != 0) {

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

@@ -4663,7 +4663,7 @@ void setCursor () {
 	* is IDC_ARROW.
 	*/
 	Cursor cursor = findCursor ();
-	long hCursor = cursor == null ? OS.LoadCursor (0, OS.IDC_ARROW) : cursor.handle;
+	long hCursor = cursor == null ? OS.LoadCursor (0, OS.IDC_ARROW) : Cursor.win32_getHandle(cursor, getZoom());
 	OS.SetCursor (hCursor);
 }