Adding loadImageData(zoom) to the AbstractImageProvider Wrapper

This refactoring is required to load unscaled ImageData for all wrappers
The loadImageData for the added functions would have similar logic to
newImageData() except the imageData returned would not be scaled but
would be an ElementAtZoom

Author: arunjose696

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

@@ -1887,6 +1887,11 @@ <T> T applyUsingAnyHandle(Function<ImageHandle, T> function) {
 	return function.apply(zoomLevelToImageHandle.values().iterator().next());
 }
 
+private int getClosestAvailableZoom(int zoom) {
+	TreeSet<Integer> availableZooms = new TreeSet<>(zoomLevelToImageHandle.keySet());
+	return Optional.ofNullable(availableZooms.higher(zoom)).orElse(availableZooms.lower(zoom));
+}
+
 /**
  * Invokes platform specific functionality to allocate a new image.
  * <p>
@@ -1939,13 +1944,14 @@ public Collection<Integer> getPreservedZoomLevels() {
 		return Collections.emptySet();
 	}
 
+	protected abstract ElementAtZoom<ImageData> loadImageData(int zoom);
+
 	abstract ImageData newImageData(ZoomContext zoomContext);
 
 	abstract AbstractImageProviderWrapper createCopy(Image image);
 
 	ImageData getScaledImageData (int zoom) {
-		TreeSet<Integer> availableZooms = new TreeSet<>(zoomLevelToImageHandle.keySet());
-		int closestZoom = Optional.ofNullable(availableZooms.higher(zoom)).orElse(availableZooms.lower(zoom));
+		int closestZoom = getClosestAvailableZoom(zoom);
 		return DPIUtil.scaleImageData(device, getImageMetadata(new ZoomContext(closestZoom)).getImageData(), zoom, closestZoom);
 	}
 
@@ -2004,12 +2010,18 @@ AbstractImageProviderWrapper createCopy(Image image) {
 	public Collection<Integer> getPreservedZoomLevels() {
 		return Collections.singleton(zoomForHandle);
 	}
+
+	@Override
+	protected ElementAtZoom<ImageData> loadImageData(int zoom) {
+		int closestZoom = getClosestAvailableZoom(zoom);
+		ImageData imageData = getImageMetadata(new ZoomContext(closestZoom)).getImageData();
+		return new ElementAtZoom<>(imageData, closestZoom);
+	}
 }
 
 private abstract class ImageFromImageDataProviderWrapper extends AbstractImageProviderWrapper {
 	private final Map<Integer, ImageData> cachedImageData = new HashMap<>();
 
-	protected abstract ElementAtZoom<ImageData> loadImageData(int zoom);
 
 	void initImage() {
 		// As the init call configured some Image attributes (e.g. type)
@@ -2180,6 +2192,20 @@ protected Rectangle getBounds(int zoom) {
 		return Win32DPIUtils.pointToPixel(rectangle, zoom);
 	}
 
+	private ImageData resolveBaseOrNewImage(ZoomContext zoomContext) {
+		int targetZoom = zoomContext.targetZoom();
+		if (zoomLevelToImageHandle.isEmpty()) {
+			return createBaseHandle(targetZoom).getImageData();
+		}
+		// if a GC is initialized with an Image (memGC != null), the image data must not be resized, because it would
+		// be a destructive operation. Therefor, a new handle is created for the requested zoom
+		if (memGC != null) {
+			return newImageHandle(zoomContext).getImageData();
+		}
+
+		return null;
+	}
+
 	@Override
 	ImageData newImageData(ZoomContext zoomContext) {
 		int targetZoom = zoomContext.targetZoom();
@@ -2194,6 +2220,12 @@ ImageData newImageData(ZoomContext zoomContext) {
 		return getScaledImageData(targetZoom);
 	}
 
+	@Override
+	protected ElementAtZoom<ImageData> loadImageData(int zoom) {
+		int closestZoom = getClosestAvailableZoom(zoom);
+		return new ElementAtZoom<>(getImageMetadata(new ZoomContext(closestZoom)).getImageData(), closestZoom);
+	}
+
 	@Override
 	protected ImageHandle newImageHandle(ZoomContext zoomContext) {
 		int targetZoom = zoomContext.targetZoom();
@@ -2326,7 +2358,6 @@ private ImageHandle initializeHandleFromSource(int zoom) {
 		return init(imageData, zoom);
 	}
 
-	protected abstract ElementAtZoom<ImageData> loadImageData(int zoom);
 
 	@Override
 	protected Rectangle getBounds(int zoom) {
@@ -2624,6 +2655,11 @@ ImageData newImageData(ZoomContext zoomContext) {
 		return getImageMetadata(zoomContext).getImageData();
 	}
 
+	@Override
+	protected ElementAtZoom<ImageData> loadImageData(int zoom) {
+		return new ElementAtZoom<>(newImageData(new ZoomContext(zoom)), zoom);
+	}
+
 	@Override
 	protected ImageHandle newImageHandle(ZoomContext zoomContext) {
 		currentZoom = zoomContext;