Do not suspend JobManager during startup

Because of possible deadlock when Javaeditor from JDT calls PDE calls
OOMPH calls P2 to download new target platform waiting for a Job that is
not executed because JobManager is suspendend

eclipse-pde/eclipse.pde#1481

Was already suggested in
https://bugs.eclipse.org/bugs/show_bug.cgi?id=514090

Author: Jörg Kubitz

bundles/org.eclipse.ui.ide.application/src/org/eclipse/ui/internal/ide/application/IDEApplication.java

@@ -43,7 +43,6 @@
 import org.eclipse.core.runtime.Platform;
 import org.eclipse.core.runtime.Status;
 import org.eclipse.core.runtime.URIUtil;
-import org.eclipse.core.runtime.jobs.Job;
 import org.eclipse.core.runtime.preferences.ConfigurationScope;
 import org.eclipse.equinox.app.IApplication;
 import org.eclipse.equinox.app.IApplicationContext;
@@ -142,12 +141,6 @@ public IDEApplication() {
 
 	@Override
 	public Object start(IApplicationContext appContext) throws Exception {
-		// Suspend the job manager to prevent background jobs from running. This
-		// is done to reduce resource contention during startup.
-		// The job manager will be resumed by the
-		// IDEWorkbenchAdvisor.postStartup method.
-		Job.getJobManager().suspend();
-
 		Display display = createDisplay();
 		// processor must be created before we start event loop
 		DelayedEventsProcessor processor = new DelayedEventsProcessor(display);

bundles/org.eclipse.ui.ide.application/src/org/eclipse/ui/internal/ide/application/IDEWorkbenchAdvisor.java

@@ -273,7 +273,6 @@ public void preStartup() {
 
 	@Override
 	public void postStartup() {
-		try {
 			refreshFromLocal();
 			activateProxyService();
 			((Workbench) PlatformUI.getWorkbench()).registerService(
@@ -283,11 +282,6 @@ public void postStartup() {
 			initializeSettingsChangeListener();
 			Display.getCurrent().addListener(SWT.Settings,
 					settingsChangeListener);
-		} finally {
-			// Resume the job manager to allow background jobs to run.
-			// The job manager was suspended by the IDEApplication.start method.
-			Job.getJobManager().resume();
-		}
 	}
 
 	/**