Track and dispose resources created by operations in GC

Each Operation subclass now maintains its own list of disposable SWT
resources. These are added to a `disposables` list within
the Operation and are disposed when the GC is disposed. This ensures
cleanup of resources.

Author: arunjose696

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

@@ -497,7 +497,8 @@ private class CopyAreaToImageOperation extends Operation {
 	private final int y;
 
 	CopyAreaToImageOperation(Image image, int x, int y) {
-		this.image = image;
+		this.image = new Image(image.getDevice(), image, SWT.IMAGE_COPY);
+		track(this.image);
 		this.x = x;
 		this.y = y;
 	}
@@ -1018,7 +1019,8 @@ private class DrawImageOperation extends Operation {
 	private final Point location;
 
 	DrawImageOperation(Image image, Point location) {
-		this.image = image;
+		this.image = new Image(image.getDevice(), image, SWT.IMAGE_COPY);
+		track(this.image);
 		this.location = location;
 	}
 
@@ -1087,7 +1089,8 @@ private class DrawScalingImageToImageOperation extends Operation {
 	private final Rectangle destination;
 
 	DrawScalingImageToImageOperation(Image image, Rectangle source, Rectangle destination) {
-		this.image = image;
+		this.image = new Image(image.getDevice(), image, SWT.IMAGE_COPY);
+		track(this.image);
 		this.source = source;
 		this.destination = destination;
 	}
@@ -1161,7 +1164,8 @@ private class DrawImageToImageOperation extends Operation {
 	private final boolean simple;
 
 	DrawImageToImageOperation(Image image, Rectangle source, Rectangle destination, boolean simple) {
-		this.image = image;
+		this.image = new Image(image.getDevice(), image, SWT.IMAGE_COPY);
+		track(this.image);
 		this.source = source;
 		this.destination = destination;
 		this.simple = simple;
@@ -1958,14 +1962,20 @@ private class DrawPathOperation extends Operation {
 	@Override
 	void apply() {
 		Path path = new Path(device, pathData);
-		long pathHandle = path.getHandle(getZoom());
-		if (pathHandle == 0) SWT.error(SWT.ERROR_INVALID_ARGUMENT);
-		initGdip();
-		checkGC(DRAW);
-		long gdipGraphics = data.gdipGraphics;
-		Gdip.Graphics_TranslateTransform(gdipGraphics, data.gdipXOffset, data.gdipYOffset, Gdip.MatrixOrderPrepend);
-		Gdip.Graphics_DrawPath(gdipGraphics, data.gdipPen, pathHandle);
-		Gdip.Graphics_TranslateTransform(gdipGraphics, -data.gdipXOffset, -data.gdipYOffset, Gdip.MatrixOrderPrepend);
+		try {
+			long pathHandle = path.getHandle(getZoom());
+			if (pathHandle == 0)
+				SWT.error(SWT.ERROR_INVALID_ARGUMENT);
+			initGdip();
+			checkGC(DRAW);
+			long gdipGraphics = data.gdipGraphics;
+			Gdip.Graphics_TranslateTransform(gdipGraphics, data.gdipXOffset, data.gdipYOffset, Gdip.MatrixOrderPrepend);
+			Gdip.Graphics_DrawPath(gdipGraphics, data.gdipPen, pathHandle);
+			Gdip.Graphics_TranslateTransform(gdipGraphics, -data.gdipXOffset, -data.gdipYOffset,
+					Gdip.MatrixOrderPrepend);
+		} finally {
+			path.dispose();
+		}
 	}
 }
 
@@ -3272,13 +3282,18 @@ private class FillPathOperation extends Operation {
 	@Override
 	void apply() {
 		Path path = new Path(device, pathData);
-		long pathHandle = path.getHandle(getZoom());
-		if (pathHandle == 0) SWT.error(SWT.ERROR_INVALID_ARGUMENT);
-		initGdip();
-		checkGC(FILL);
-		int mode = OS.GetPolyFillMode(handle) == OS.WINDING ? Gdip.FillModeWinding : Gdip.FillModeAlternate;
-		Gdip.GraphicsPath_SetFillMode(pathHandle, mode);
-		Gdip.Graphics_FillPath(data.gdipGraphics, data.gdipBrush, pathHandle);
+		try {
+			long pathHandle = path.getHandle(getZoom());
+			if (pathHandle == 0)
+				SWT.error(SWT.ERROR_INVALID_ARGUMENT);
+			initGdip();
+			checkGC(FILL);
+			int mode = OS.GetPolyFillMode(handle) == OS.WINDING ? Gdip.FillModeWinding : Gdip.FillModeAlternate;
+			Gdip.GraphicsPath_SetFillMode(pathHandle, mode);
+			Gdip.Graphics_FillPath(data.gdipGraphics, data.gdipBrush, pathHandle);
+		} finally {
+			path.dispose();
+		}
 	}
 }
 
@@ -4629,7 +4644,9 @@ private class SetBackgroundOperation extends Operation {
 	private final Color color;
 
 	SetBackgroundOperation(Color color) {
-		this.color = color;
+		RGB rgb = color.getRGB();
+        this.color = new Color(color.getDevice(), rgb);
+        track(this.color);
 	}
 
 	@Override
@@ -4676,6 +4693,7 @@ private class SetBackgroundPatternOperation extends Operation {
 
 	SetBackgroundPatternOperation(Pattern pattern) {
 		this.pattern = pattern == null ? null : pattern.copy();
+		track(this.pattern);
 	}
 
 	@Override
@@ -4938,7 +4956,8 @@ private class SetFontOperation extends Operation {
 	private final Font font;
 
 	SetFontOperation(Font font) {
-		this.font = font;
+		this.font = new Font(font.getDevice(), font.getFontData());
+		track(this.font);
 	}
 
 	@Override
@@ -4973,7 +4992,9 @@ private class SetForegroundOperation extends Operation {
 	private final Color color;
 
 	SetForegroundOperation(Color color) {
-		this.color = color;
+		RGB rgb = color.getRGB();
+		this.color = new Color(color.getDevice(), rgb);
+        track(this.color);
 	}
 
 	@Override
@@ -5019,6 +5040,7 @@ private class SetForegroundPatternOperation extends Operation {
 
 	SetForegroundPatternOperation(Pattern pattern) {
 		this.pattern = pattern == null ? null : pattern.copy();
+		track(this.pattern);
 	}
 
 	@Override
@@ -5604,6 +5626,7 @@ private class SetTransformOperation extends Operation {
 			float[] elements = new float[6];
 			transform.getElements(elements);
 			this.transform = new Transform(device, elements[0], elements[1], elements[2], elements[3], elements[4], elements[5]);
+			track(this.transform);
 		} else {
 			this.transform = null;
 		}
@@ -5876,8 +5899,36 @@ private void createGcHandle(Drawable drawable, GCData newData, int nativeZoom) {
 	}
 }
 
+
+@Override
+public void dispose() {
+    super.dispose();
+    disposeOperations();
+}
+
+private void disposeOperations() {
+    for (Operation op : operations) {
+        op.disposeAll();
+    }
+    operations.clear();
+}
+
 private abstract class Operation {
+	private final List<Resource> disposables = new ArrayList<>();
 	abstract void apply();
+
+	protected void track(Resource resource) {
+		if (resource != null) {
+			disposables.add(resource);
+		}
+	}
+
+	void disposeAll() {
+		for (Resource r : disposables) {
+			r.dispose();
+		}
+		disposables.clear();
+	}
 }
 }