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.WithMonitor 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;
 	}
 
@@ -59,6 +59,19 @@ private SingleZoomCoordinateSystemMapper getSingleZoomCoordinateSystemMapper() {
 		return new SingleZoomCoordinateSystemMapper(null);
 	}
 
+	@Test
+	void mapMonitorBoundsShouldBeSameForAllMonitorsWithSingleZoomCoordinateMapper() {
+		CoordinateSystemMapper mapper = getSingleZoomCoordinateSystemMapper();
+		setupMonitors(mapper);
+		Rectangle.WithMonitor rectInPixelsAt100Zoom = new Rectangle.WithMonitor(2100, 100, 500, 500, monitors[1]);
+		Rectangle rectAfterMappingAt100 = mapper.mapMonitorBounds(rectInPixelsAt100Zoom);
+
+		Rectangle.WithMonitor rectInPixelsAt200Zoom = new Rectangle.WithMonitor(2100, 100, 500, 500, monitors[0]);
+		Rectangle rectAfterMappingAt200 = mapper.mapMonitorBounds(rectInPixelsAt200Zoom);
+
+		assertEquals(rectAfterMappingAt100,rectAfterMappingAt200);
+	}
+
 	@ParameterizedTest
 	@MethodSource("provideCoordinateSystemMappers")
 	void translatePointInNoMonitorBackAndForthShouldBeTheSame(CoordinateSystemMapper mapper) {

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.WithMonitor rectangle);
 
 	Point translateFromDisplayCoordinates(Point point);
 

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

@@ -2225,16 +2225,10 @@ 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;
@@ -2244,6 +2238,12 @@ Monitor getMonitor (long hmonitor) {
 	 * to scaling issue on OS Win8.1 and above, for more details refer bug 537614.
 	 */
 	monitor.zoom = result;
+	Rectangle.WithMonitor boundsInPixels = new Rectangle.WithMonitor(lpmi.rcMonitor_left, lpmi.rcMonitor_top, lpmi.rcMonitor_right - lpmi.rcMonitor_left,lpmi.rcMonitor_bottom - lpmi.rcMonitor_top, monitor);
+	Rectangle.WithMonitor 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));
+
 	return monitor;
 }
 

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

@@ -84,7 +84,9 @@ public Rectangle map(Control from, Control to, int x, int y, int width, int heig
 	}
 
 	@Override
-	public Rectangle mapMonitorBounds(Rectangle rect, int zoom) {
+	public Rectangle mapMonitorBounds(Rectangle.WithMonitor rect) {
+		Monitor monitor = rect.getMonitor();
+		int zoom = getApplicableMonitorZoom(monitor);
 		Rectangle bounds = Win32DPIUtils.pixelToPoint(rect, zoom);
 		bounds.x = rect.x;
 		bounds.y = rect.y;

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.WithMonitor rect) {
+		return Win32DPIUtils.pixelToPoint(rect, DPIUtil.getDeviceZoom());
 	}
 
 	@Override