Reintroduce scaling methods for Rectangle

This commit reintroduces scaling methods for plain Rectangle to provide
backwards compatibility to the consumers which explicitly create a
Rectangle and want to call scaleBounds or scaleUp/scaleDown since they
have different behaviour and Rectangle.OfFloat unifies this.

Author: amartya4256

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

@@ -100,11 +100,11 @@ public void testCorrectScaleUpUsingDifferentSetBoundsMethod() {
 
 		button.setBounds(new Rectangle(0, 47, 200, 47));
 		assertEquals("Control::setBounds(Rectangle) doesn't scale up correctly",
-				new Rectangle(0, 82, 350, 82), button.getBoundsInPixels());
+				new Rectangle(0, 82, 350, 83), button.getBoundsInPixels());
 
 		button.setBounds(0, 47, 200, 47);
 		assertEquals("Control::setBounds(int, int, int, int) doesn't scale up correctly",
-				new Rectangle(0, 82, 350, 82), button.getBoundsInPixels());
+				new Rectangle(0, 82, 350, 83), button.getBoundsInPixels());
 	}
 
 	record FontComparison(int originalFontHeight, int currentFontHeight) {

bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java

@@ -255,6 +255,20 @@ public static Rectangle autoScaleDown(Rectangle rect) {
 }
 
 public static Rectangle scaleDown(Rectangle rect, int zoom) {
+	if (zoom == 100 || rect == null) return rect;
+	if (rect instanceof Rectangle.OfFloat rectOfFloat) return scaleDown(rectOfFloat, zoom);
+	Rectangle scaledRect = new Rectangle.OfFloat (0,0,0,0);
+	Point scaledTopLeft = scaleDown(new Point (rect.x, rect.y), zoom);
+	Point scaledBottomRight = scaleDown(new Point (rect.x + rect.width, rect.y + rect.height), zoom);
+
+	scaledRect.x = scaledTopLeft.x;
+	scaledRect.y = scaledTopLeft.y;
+	scaledRect.width = scaledBottomRight.x - scaledTopLeft.x;
+	scaledRect.height = scaledBottomRight.y - scaledTopLeft.y;
+	return scaledRect;
+}
+
+public static Rectangle scaleDown(Rectangle.OfFloat rect, int zoom) {
 	return scaleBounds(rect, 100, zoom);
 }
 /**
@@ -323,6 +337,21 @@ public static boolean isSmoothScalingEnabled() {
  * Returns a new rectangle as per the scaleFactor.
  */
 public static Rectangle scaleBounds (Rectangle rect, int targetZoom, int currentZoom) {
+	if (rect == null || targetZoom == currentZoom) return rect;
+	if (rect instanceof Rectangle.OfFloat rectOfFloat) return scaleBounds(rectOfFloat, targetZoom, currentZoom);
+	float scaleFactor = ((float)targetZoom) / (float)currentZoom;
+	Rectangle returnRect = new Rectangle.OfFloat (0,0,0,0);
+	returnRect.x = Math.round (rect.x * scaleFactor);
+	returnRect.y = Math.round (rect.y * scaleFactor);
+	returnRect.width = Math.round (rect.width * scaleFactor);
+	returnRect.height = Math.round (rect.height * scaleFactor);
+	return returnRect;
+}
+
+/**
+ * Returns a new rectangle as per the scaleFactor.
+ */
+public static Rectangle scaleBounds (Rectangle.OfFloat rect, int targetZoom, int currentZoom) {
 	if (rect == null || targetZoom == currentZoom) return rect;
 	Rectangle.OfFloat fRect = FloatAwareGeometryFactory.createFrom(rect);
 	float scaleFactor = getScalingFactor(targetZoom, currentZoom);
@@ -454,6 +483,20 @@ public static Rectangle autoScaleUp(Rectangle rect) {
 }
 
 public static Rectangle scaleUp(Rectangle rect, int zoom) {
+	if (zoom == 100 || rect == null) return rect;
+	if (rect instanceof Rectangle.OfFloat rectOfFloat) return scaleUp(rectOfFloat, zoom);
+	Rectangle scaledRect = new Rectangle.OfFloat(0,0,0,0);
+	Point scaledTopLeft = scaleUp (new Point(rect.x, rect.y), zoom);
+	Point scaledBottomRight = scaleUp (new Point(rect.x + rect.width, rect.y + rect.height), zoom);
+
+	scaledRect.x = scaledTopLeft.x;
+	scaledRect.y = scaledTopLeft.y;
+	scaledRect.width = scaledBottomRight.x - scaledTopLeft.x;
+	scaledRect.height = scaledBottomRight.y - scaledTopLeft.y;
+	return scaledRect;
+}
+
+public static Rectangle scaleUp(Rectangle.OfFloat rect, int zoom) {
 	return scaleBounds(rect, zoom, 100);
 }
 

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/DPIUtilTests.java

@@ -296,7 +296,7 @@ public void scaleUpPoint() {
 	@Test
 	public void scaleUpRectangle() {
 		Rectangle valueAt200 = new Rectangle(100, 150, 10, 14);
-		Rectangle valueAt150 = new Rectangle(75, 113, 8, 11);
+		Rectangle valueAt150 = new Rectangle(75, 113, 8, 10);
 		Rectangle valueAt100 = new Rectangle(50, 75, 5, 7);
 
 		Rectangle scaledValue = DPIUtil.autoScaleUp(valueAt100);
@@ -321,12 +321,12 @@ public void scaleUpRectangle() {
 	}
 
 	@Test
-	public void scaleDownscaleUpRectangleInvertible() {
+	public void scaleDownscaleUpRectangleOfFloatInvertible() {
 		int[] zooms = new int[] {25, 50, 75, 100, 125, 150, 175, 200, 225, 250, 275, 300, 325, 350, 375, 400};
 		for (int zoom1 : zooms) {
 			for (int zoom2 : zooms) {
 				for (int i = 1; i <= 10000; i++) {
-					Rectangle rect = new Rectangle(0, 0, i, i);
+					Rectangle rect = new Rectangle.OfFloat(0, 0, i, i);
 					Rectangle scaleDown = DPIUtil.scaleDown(rect, zoom1);
 					Rectangle scaleUp = DPIUtil.scaleUp(scaleDown, zoom2);
 					scaleDown = DPIUtil.scaleDown(scaleUp, zoom2);