Fix stack overflow in DPIUtil.autoScaleImageData due to

ImageDataProvider zoom handling

In PR eclipse-platform#2249, a strict check was introduced  that ImageData should be
linearly scaled for other zoom levels. And imageDataProviders in SWT
repo not following this strict checks were modified (eg to return only
imageData at 100%).

However, this change broke DPIUtil.autoScaleImageData. That function
creates an Image using the system zoom level, draws it on a GC at scaled
dimensions, and then requests ImageData at 100% zoom from the image.
Since the ImageDataProvider was now restricted to 100% zoom only,
requesting imageData at other zoom levels while creating an image at
system zoom level triggered recursive calls,  causing a stack overflow
error.

The current commit reverts the behavior of the ImageDataProvider used in
DPIUtil.autoScaleImageData to return the same ImageData at all zoom
levels.
We also Disable the strict zoom check in this specific case, because the
scaling is done by GC.drawImage() and the ImageData used to create the
is expected to be same.

Author: arunjose696

bundles/org.eclipse.swt/Eclipse SWT/cocoa/org/eclipse/swt/graphics/Image.java

@@ -1811,22 +1811,26 @@ public String toString () {
  * API for Image. It is marked public only so that it
  * can be shared within the packages provided by SWT.
  *
- * Draws a scaled image using the GC by another image.
+ * Draws a scaled image using the GC for a given imageData.
  *
  * @param gc the GC to draw on the resulting image
- * @param original the image which is supposed to be scaled and drawn on the resulting image
+ * @param imageData the imageData which is used to draw the scaled Image
  * @param width the width of the original image
  * @param height the height of the original image
  * @param scaleFactor the factor with which the image is supposed to be scaled
  *
  * @noreference This method is not intended to be referenced by clients.
  */
-public static void drawScaled(GC gc, Image original, int width, int height, float scaleFactor) {
-	gc.drawImage (original, 0, 0, CocoaDPIUtil.pixelToPoint (width), CocoaDPIUtil.pixelToPoint (height),
+public static void drawScaled(GC gc, ImageData imageData, int width, int height, float scaleFactor) {
+	boolean strictCheckValue = Device.strictChecks;
+	Device.strictChecks = false;
+	Image imageToDraw = new Image(gc.device, (ImageDataProvider) zoom ->  imageData);
+	gc.drawImage (imageToDraw, 0, 0, CocoaDPIUtil.pixelToPoint (width), CocoaDPIUtil.pixelToPoint (height),
 			/* E.g. destWidth here is effectively DPIUtil.autoScaleDown (scaledWidth), but avoiding rounding errors.
 			 * Nevertheless, we still have some rounding errors due to the point-based API GC#drawImage(..).
 			 */
 			0, 0, Math.round (CocoaDPIUtil.pixelToPoint (width * scaleFactor)), Math.round (CocoaDPIUtil.pixelToPoint (height * scaleFactor)));
+	Device.strictChecks = strictCheckValue;
 }
 
 private final class CocoaDPIUtil {

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

@@ -153,12 +153,11 @@ public static ImageData autoScaleImageData (Device device, final ImageData image
 	int defaultZoomLevel = 100;
 	boolean useSmoothScaling = isSmoothScalingEnabled() && imageData.getTransparencyType() != SWT.TRANSPARENCY_MASK;
 	if (useSmoothScaling) {
-		Image original = new Image(device, (ImageDataProvider) zoom -> (zoom == defaultZoomLevel) ? imageData : null);
 		ImageGcDrawer drawer =  new ImageGcDrawer() {
 			@Override
 			public void drawOn(GC gc, int imageWidth, int imageHeight) {
 				gc.setAntialias (SWT.ON);
-				Image.drawScaled(gc, original, width, height, scaleFactor);
+				Image.drawScaled(gc, imageData, width, height, scaleFactor);
 			};
 
 			@Override
@@ -168,7 +167,6 @@ public int getGcStyle() {
 		};
 		Image resultImage = new Image (device, drawer, scaledWidth, scaledHeight);
 		ImageData result = resultImage.getImageData (defaultZoomLevel);
-		original.dispose ();
 		resultImage.dispose ();
 		return result;
 	} else {

bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/graphics/Image.java

@@ -1574,22 +1574,26 @@ public String toString () {
  * API for Image. It is marked public only so that it
  * can be shared within the packages provided by SWT.
  *
- * Draws a scaled image using the GC by another image.
+ * Draws a scaled image using the GC for a given imageData.
  *
  * @param gc the GC to draw on the resulting image
- * @param original the image which is supposed to be scaled and drawn on the resulting image
+ * @param imageData the imageData which is used to draw the scaled Image
  * @param width the width of the original image
  * @param height the height of the original image
  * @param scaleFactor the factor with which the image is supposed to be scaled
  *
  * @noreference This method is not intended to be referenced by clients.
  */
-public static void drawScaled(GC gc, Image original, int width, int height, float scaleFactor) {
-	gc.drawImage (original, 0, 0, width, height,
+public static void drawScaled(GC gc, ImageData imageData, int width, int height, float scaleFactor) {
+	boolean originalStrictChecks = Device.strictChecks;
+	Device.strictChecks = false;
+	Image imageToDraw = new Image(gc.device, (ImageDataProvider) zoom -> imageData);
+	gc.drawImage (imageToDraw, 0, 0, width, height,
 			/* E.g. destWidth here is effectively DPIUtil.autoScaleDown (scaledWidth), but avoiding rounding errors.
 			 * Nevertheless, we still have some rounding errors due to the point-based API GC#drawImage(..).
 			 */
 			0, 0, Math.round (width * scaleFactor), Math.round (height * scaleFactor));
+	Device.strictChecks = originalStrictChecks;
 }
 
 private final class GtkDPIUtil {

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

@@ -839,19 +839,23 @@ public static long win32_getHandle (Image image, int zoom) {
  * API for Image. It is marked public only so that it
  * can be shared within the packages provided by SWT.
  *
- * Draws a scaled image using the GC by another image.
+ * Draws a scaled image using the GC for a given imageData.
  *
  * @param gc the GC to draw on the resulting image
- * @param original the image which is supposed to be scaled and drawn on the resulting image
+ * @param imageData the imageData which is used to draw the scaled Image
  * @param width the width of the original image
  * @param height the height of the original image
  * @param scaleFactor the factor with which the image is supposed to be scaled
  *
  * @noreference This method is not intended to be referenced by clients.
  */
-public static void drawScaled(GC gc, Image original, int width, int height, float scaleFactor) {
-	gc.drawImage (original, 0, 0, width, height,
+public static void drawScaled(GC gc, ImageData imageData, int width, int height, float scaleFactor) {
+	boolean originalStrictChecks = Device.strictChecks;
+	Device.strictChecks = false;
+	Image imageToDraw = new Image(gc.device, (ImageDataProvider) zoom ->  imageData);
+	gc.drawImage (imageToDraw, 0, 0, width, height,
 			0, 0, Math.round (width * scaleFactor), Math.round (height * scaleFactor), false);
+	Device.strictChecks = originalStrictChecks;
 }
 
 long [] createGdipImage(Integer zoom) {