Remove unnecessary disablement of CloseTestWindowsRule in leak test

UITestCase provides a method to disable window tracking of the
CloseTestWindowsRule, which is only used at a single place that does not
even require it.

This change removes the methods and the according usage.

Author: HeikoKlare

tests/org.eclipse.ui.tests.harness/src/org/eclipse/ui/tests/harness/util/UITestCase.java

@@ -154,12 +154,4 @@ protected void doTearDown() throws Exception {
 		closeTestWindows.after();
 	}
 
-	/**
-	 * Set whether the window listener will manage opening and closing of created windows.
-	 */
-	protected void manageWindows(boolean manage) {
-		closeTestWindows.setEnabled(manage);
-	}
-
-
 }

tests/org.eclipse.ui.tests/Eclipse UI Tests/org/eclipse/ui/tests/leaks/LeakTests.java

@@ -285,25 +285,17 @@ public void testTextEditorContextMenu() throws Exception {
 
 	@Test
 	public void testSimpleWindowLeak() throws Exception {
-		// turn off window management so that we dont have a reference to our
-		// new
-		// window in the listener
-		manageWindows(false);
+		ReferenceQueue queue = new ReferenceQueue();
+		IWorkbenchWindow newWindow = openTestWindow();
+
+		assertNotNull(newWindow);
+		Reference ref = createReference(queue, newWindow);
 		try {
-			ReferenceQueue queue = new ReferenceQueue();
-			IWorkbenchWindow newWindow = openTestWindow();
-
-			assertNotNull(newWindow);
-			Reference ref = createReference(queue, newWindow);
-			try {
-				newWindow.close();
-				newWindow = null;
-				checkRef(queue, ref);
-			} finally {
-				ref.clear();
-			}
+			newWindow.close();
+			newWindow = null;
+			checkRef(queue, ref);
 		} finally {
-			manageWindows(true);
+			ref.clear();
 		}
 	}