Bug 409969 - Target Platform GC should run more often

Change-Id: I06370c3b9e6316392fc10ba7b417cd46763cc5da

1 files changed, 3 insertions, 5 deletions

diff --git a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/preferences/TargetPlatformPreferencePage.java b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/preferences/TargetPlatformPreferencePage.java
index a9c2bb07e9..ad4a8f97ff 100644
--- a/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/preferences/TargetPlatformPreferencePage.java
+++ b/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/preferences/TargetPlatformPreferencePage.java
@@ -815,6 +815,9 @@ public class TargetPlatformPreferencePage extends PreferencePage implements IWor
 
 			LoadTargetDefinitionJob.load(toLoad, listener);
 			fPrevious = toLoad == null ? null : toLoad;
+
+			// Start a separate job to clean p2 bundle pool
+			runGC();
 		} else {
 			// Manually update the active target and status line to update name, resolve status, and errors
 			if (fActiveTarget != null) {
@@ -824,17 +827,12 @@ public class TargetPlatformPreferencePage extends PreferencePage implements IWor
 		}
 
 		fMoved.clear();
-		boolean gc = !fRemoved.isEmpty();
 		fRemoved.clear();
 		if (toLoad != null) {
 			fActiveTarget = toLoad;
 		}
 		fTableViewer.refresh(true);
 		updateButtons();
-		// start job to do GC
-		if (gc) {
-			runGC();
-		}
 		return super.performOk();
 	}