New test for Clipboard

Testing the Clipboard can be difficult because there are a lot of system
issues that can interfere, but of particular concern are:

1. System clipboard managers which may take ownership of the clipboard
at unexpected times.
2. Limitations as to when processes can access clipboard, such as on Wayland
where only active window can access clipboard.
3. Different behaviour when copying within a single process than between
processes.

These tests aim to resolve these issues.

For the system clipboard manager there are a lot of extra sleep calls
to allow clipboard manager to complete operations before continuing tests.
In addition, we run all the tests multiple times by default to ensure
stability.

For the process limitations, we carefully control when the shell is created
because we often cannot get focus back when shell ends up in the background.
See the openAndFocusShell and openAndFocusRemote methods.

For the different behaviours, we spin up a simple Swing app in a new
process (the "remote" in openAndFocusRemote above). This app can be
directed, over RMI, to access the clipboard. This allows our test to
place data on the clipboard and ensure that the remote app can read the
data successfully.

For now this test only covers basic text (and a little of RTF). Adding
Image and other transfers is part of the future work as such functionality
is added in GTK4 while working on #2126

For the changes to SwtTestUtil that we required:

1. isGTK4 moved from Test_org_eclipse_swt_widgets_Shell.java to the Utils
2. processEvents was limited to 20 readAndDispatch calls per second due
to the ordering of the targetTimestamp check. This change allows full speed
readAndDispatching.
3. getPath was refactored to allow better control of source and destination
of files extracted. See extracting of class files for remote Swing app
in startRemoteClipboardCommands method

Part of #2126
Split out of #2538

Author: jonahgraham

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

@@ -47,7 +47,8 @@
 		Test_org_eclipse_swt_accessibility_AccessibleTextEvent.class,
 		Test_org_eclipse_swt_internal_SVGRasterizer.class,
 		DPIUtilTests.class,
-		JSVGRasterizerTest.class})
+		JSVGRasterizerTest.class,
+		Test_org_eclipse_swt_dnd_Clipboard.class,})
 public class AllNonBrowserTests {
 	private static List<Error> leakedResources;
 

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

@@ -105,7 +105,8 @@ public class SwtTestUtil {
 
 	public final static boolean isX11 = isGTK
 			&& "x11".equals(System.getProperty("org.eclipse.swt.internal.gdk.backend"));
-
+	public final static boolean isGTK4 = isGTK
+			&& System.getProperty("org.eclipse.swt.internal.gtk.version", "").startsWith("4");
 
 	/**
 	 * The palette used by images. See {@link #getAllPixels(Image)} and {@link #createImage}
@@ -400,13 +401,16 @@ public static void processEvents(int timeoutMs, BooleanSupplier breakCondition)
 	long targetTimestamp = System.currentTimeMillis() + timeoutMs;
 	Display display = Display.getCurrent();
 	while (!breakCondition.getAsBoolean()) {
-		if (!display.readAndDispatch()) {
-			if (System.currentTimeMillis() < targetTimestamp) {
-				Thread.sleep(50);
-			} else {
+		while (display.readAndDispatch()) {
+			if (System.currentTimeMillis() >= targetTimestamp) {
 				return;
 			}
 		}
+		if (System.currentTimeMillis() < targetTimestamp) {
+			Thread.sleep(50);
+		} else {
+			return;
+		}
 	}
 }
 
@@ -583,18 +587,24 @@ public static boolean hasPixelNotMatching(Image image, Color nonMatchingColor, R
 }
 
 public static Path getPath(String fileName, TemporaryFolder tempFolder) {
-	Path filePath = tempFolder.getRoot().toPath().resolve("image-resources").resolve(Path.of(fileName));
-	if (!Files.isRegularFile(filePath)) {
+	Path path = tempFolder.getRoot().toPath();
+	Path filePath = path.resolve("image-resources").resolve(Path.of(fileName));
+	return getPath(fileName, filePath);
+}
+
+public static Path getPath(String sourceFilename, Path destinationPath) {
+	if (!Files.isRegularFile(destinationPath)) {
 		// Extract resource on the classpath to a temporary file to ensure it's
 		// available as plain file, even if this bundle is packed as jar
-		try (InputStream inStream = SwtTestUtil.class.getResourceAsStream(fileName)) {
-			assertNotNull(inStream, "InputStream == null for file " + fileName);
-			Files.createDirectories(filePath.getParent());
-			Files.copy(inStream, filePath);
+		try (InputStream inStream = SwtTestUtil.class.getResourceAsStream(sourceFilename)) {
+			assertNotNull(inStream, "InputStream == null for file " + sourceFilename);
+			Files.createDirectories(destinationPath.getParent());
+			Files.copy(inStream, destinationPath);
 		} catch (IOException e) {
 			throw new IllegalArgumentException(e);
 		}
 	}
-	return filePath;
+	return destinationPath;
 }
+
 }