Remove zoom parameter from CoordinateSystemMapper#mapMonitorBounds()

The CoordinateSystemMapper#mapMonitorBounds() method currently accepts a
zoom value and that is wrong for SingleZoomCoordinateMapper since it
should always use the global zoom (DPIUtil#getDeviceZoom()) instead of
monitor zoom. This commit changes how zoom is passed through Rectangle
with monitor for MultiZoomCoordinateMapper while that monitor will be
ignored for SingleZoomCoordinateMapper.

Author: ShahzaibIbrahim

bundles/org.eclipse.swt/Eclipse SWT Tests/win32/org/eclipse/swt/widgets/CoordinateSystemMapperTests.java

@@ -20,7 +20,6 @@
 import java.util.stream.*;
 
 import org.eclipse.swt.graphics.*;
-import org.eclipse.swt.internal.*;
 import org.junit.jupiter.api.*;
 import org.junit.jupiter.params.*;
 import org.junit.jupiter.params.provider.*;
@@ -33,10 +32,11 @@ class CoordinateSystemMapperTests {
 
 	private Monitor createMonitor(CoordinateSystemMapper mapper, Rectangle boundsInPixels, int nativeZoom) {
 		Monitor monitor = new Monitor();
-		Rectangle bounds = mapper.mapMonitorBounds(boundsInPixels, DPIUtil.getZoomForAutoscaleProperty(nativeZoom));
+		monitor.zoom = nativeZoom;
+		Rectangle boundsInPixelWithMonitor = new Rectangle.WithMonitor(boundsInPixels.x, boundsInPixels.y, boundsInPixels.width, boundsInPixels.height, monitor);
+		Rectangle bounds = mapper.mapMonitorBounds(boundsInPixelWithMonitor);
 		monitor.setBounds(bounds);
 		monitor.setClientArea(bounds);
-		monitor.zoom = nativeZoom;
 		return monitor;
 	}
 

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

@@ -25,7 +25,7 @@ interface CoordinateSystemMapper {
 
 	Point map(Control from, Control to, int x, int y);
 
-	Rectangle mapMonitorBounds(Rectangle rectangle, int zoom);
+	Rectangle mapMonitorBounds(Rectangle rectangle);
 
 	Point translateFromDisplayCoordinates(Point point);
 

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

@@ -2225,25 +2225,25 @@ Monitor getMonitor (long hmonitor) {
 	OS.GetMonitorInfo (hmonitor, lpmi);
 	Monitor monitor = new Monitor ();
 	monitor.handle = hmonitor;
-	Rectangle boundsInPixels = new Rectangle(lpmi.rcMonitor_left, lpmi.rcMonitor_top, lpmi.rcMonitor_right - lpmi.rcMonitor_left,lpmi.rcMonitor_bottom - lpmi.rcMonitor_top);
-	Rectangle clientAreaInPixels = new Rectangle(lpmi.rcWork_left, lpmi.rcWork_top, lpmi.rcWork_right - lpmi.rcWork_left, lpmi.rcWork_bottom - lpmi.rcWork_top);
 	int [] dpiX = new int[1];
 	int [] dpiY = new int[1];
 	int result = OS.GetDpiForMonitor (monitor.handle, OS.MDT_EFFECTIVE_DPI, dpiX, dpiY);
 	result = (result == OS.S_OK) ? DPIUtil.mapDPIToZoom (dpiX[0]) : 100;
-
-	int autoscaleZoom = DPIUtil.getZoomForAutoscaleProperty(result);
-	monitor.setBounds(coordinateSystemMapper.mapMonitorBounds(boundsInPixels, autoscaleZoom));
-	monitor.setClientArea(coordinateSystemMapper.mapMonitorBounds(clientAreaInPixels, autoscaleZoom));
 	if (result == 0) {
 		System.err.println("***WARNING: GetDpiForMonitor: SWT could not get valid monitor scaling factor.");
 		result = 100;
 	}
+	monitor.zoom = result;
+	Rectangle boundsInPixels = new Rectangle.WithMonitor(lpmi.rcMonitor_left, lpmi.rcMonitor_top, lpmi.rcMonitor_right - lpmi.rcMonitor_left,lpmi.rcMonitor_bottom - lpmi.rcMonitor_top, monitor);
+	Rectangle clientAreaInPixels = new Rectangle.WithMonitor(lpmi.rcWork_left, lpmi.rcWork_top, lpmi.rcWork_right - lpmi.rcWork_left, lpmi.rcWork_bottom - lpmi.rcWork_top, monitor);
+
+	monitor.setBounds(coordinateSystemMapper.mapMonitorBounds(boundsInPixels));
+	monitor.setClientArea(coordinateSystemMapper.mapMonitorBounds(clientAreaInPixels));
+
 	/*
 	 * Always return true monitor zoom value as fetched from native, else will lead
 	 * to scaling issue on OS Win8.1 and above, for more details refer bug 537614.
 	 */
-	monitor.zoom = result;
 	return monitor;
 }
 

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

@@ -84,11 +84,17 @@ public Rectangle map(Control from, Control to, int x, int y, int width, int heig
 	}
 
 	@Override
-	public Rectangle mapMonitorBounds(Rectangle rect, int zoom) {
-		Rectangle bounds = Win32DPIUtils.pixelToPoint(rect, zoom);
-		bounds.x = rect.x;
-		bounds.y = rect.y;
-		return bounds;
+	public Rectangle mapMonitorBounds(Rectangle rect) {
+		if (rect instanceof Rectangle.WithMonitor rectWithMonitor) {
+			Monitor monitor = rectWithMonitor.getMonitor();
+			int zoom = getApplicableMonitorZoom(monitor);
+			Rectangle bounds = Win32DPIUtils.pixelToPoint(rect, zoom);
+			bounds.x = rect.x;
+			bounds.y = rect.y;
+			return bounds;
+		} else {
+			return rect;
+		}
 	}
 
 	@Override

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

@@ -71,8 +71,8 @@ public Rectangle map(Control from, Control to, int x, int y, int width, int heig
 	}
 
 	@Override
-	public Rectangle mapMonitorBounds(Rectangle rect, int zoom) {
-		return Win32DPIUtils.pixelToPoint(rect, zoom);
+	public Rectangle mapMonitorBounds(Rectangle rect) {
+		return Win32DPIUtils.pixelToPoint(rect, DPIUtil.getDeviceZoom());
 	}
 
 	@Override