Utilize mark and reset for stream handling

Author: Michael5601

bundles/org.eclipse.swt.svg/src/org/eclipse/swt/svg/JSVGRasterizer.java

@@ -58,11 +58,22 @@ public static void intializeJSVGRasterizer() {
 
 	@Override
 	public ImageData rasterizeSVG(InputStream stream, float scalingFactor) throws IOException {
+		if (stream == null) {
+	        throw new IllegalArgumentException("InputStream cannot be null");
+	    }
+		stream.mark(Integer.MAX_VALUE);
 		if(svgLoader == null) {
 			svgLoader = new SVGLoader();
 		}
 		SVGDocument svgDocument = null;
-		svgDocument = svgLoader.load(stream, null, LoaderContext.createDefault());
+		InputStream nonClosingStream = new FilterInputStream(stream) {
+		        @Override
+		        public void close() throws IOException {
+		            // Do nothing to prevent closing the underlying stream
+		        }
+		    };
+        svgDocument = svgLoader.load(nonClosingStream, null, LoaderContext.createDefault());
+        stream.reset();
 		if (svgDocument != null) {
 			FloatSize size = svgDocument.size();
 			double originalWidth = size.getWidth();
@@ -150,8 +161,8 @@ else if (bufferedImage.getColorModel() instanceof ComponentColorModel) {
 	    return null;
 	}
 
-	public boolean isSVGFile(InputStream inputStream) throws IOException {
-	    if (inputStream == null) {
+	public boolean isSVGFile(InputStream stream) throws IOException {
+		if (stream == null) {
 	        throw new IllegalArgumentException("InputStream cannot be null");
 	    }
 	    int firstByte = inputStream.read();

bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/graphics/SVGRasterizer.java

@@ -43,5 +43,5 @@ public interface SVGRasterizer {
 	 * @throws IOException              if an error occurs while reading the stream.
 	 * @throws IllegalArgumentException if the input stream is {@code null}.
 	 */
-	public boolean isSVGFile(InputStream inputStream) throws IOException;
+	public boolean isSVGFile(InputStream stream) throws IOException;
 }

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

@@ -178,35 +178,26 @@ public ImageData[] load(InputStream stream) {
 public ImageData[] load(InputStream stream, int zoom) {
 	if (stream == null) SWT.error(SWT.ERROR_NULL_ARGUMENT);
     reset();
-    byte[] bytes = null;
-	try {
-		bytes = stream.readAllBytes();
-	} catch (IOException e) {
-		SWT.error(SWT.ERROR_IO, e);
+    if (!stream.markSupported()) {
+		stream = new BufferedInputStream(stream);
 	}
+
 	SVGRasterizer rasterizer = SVGRasterizerRegistry.getRasterizer();
 	if (rasterizer != null && zoom != 0) {
-		try (InputStream imageStream = new ByteArrayInputStream(bytes)) {
-			if (rasterizer.isSVGFile(imageStream)) {
+		try {
+			if (rasterizer.isSVGFile(stream)) {
 				float scalingFactor = zoom / 100.0f;
-				try (InputStream svgFileStream = new ByteArrayInputStream(bytes)) {
-					ImageData rasterizedData = rasterizer.rasterizeSVG(svgFileStream, scalingFactor);
-					if (rasterizedData != null) {
-						data = new ImageData[]{rasterizedData};
-					    return data;
-					}
+				ImageData rasterizedData = rasterizer.rasterizeSVG(stream, scalingFactor);
+				if (rasterizedData != null) {
+					data = new ImageData[]{rasterizedData};
+				    return data;
 				}
 			}
 		} catch (IOException e) {
 			//ignore.
 		}
 	}
-	try (InputStream fallbackStream = new ByteArrayInputStream(bytes)) {
-		return loadDefault(fallbackStream);
-	} catch (IOException e) {
-		SWT.error(SWT.ERROR_IO, e);
-	}
-	return null;
+	return loadDefault(stream);
 }
 
 private ImageData[] loadDefault(InputStream stream) {