Catch IllegalArgumentException when resolving target definition

When the target editor is opened, PDE tries to delete the old target
handle as part of "Bug 545681 - UI Freeze of >800s in TargetEditor when
opening editor".

If the profile is not locked (due to whatever reasons), an
IllegalArgumentException is thrown, blocking the resolution of this
file.

Author: ptziegler

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/targetdefinition/TargetEditor.java

@@ -40,6 +40,7 @@
 import org.eclipse.core.runtime.IPath;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.SafeRunner;
 import org.eclipse.core.runtime.Status;
 import org.eclipse.core.runtime.jobs.Job;
 import org.eclipse.core.runtime.jobs.JobChangeAdapter;
@@ -683,12 +684,10 @@ public void contentsChanged(ITargetDefinition definition, Object source, boolean
 					@Override
 					protected IStatus run(IProgressMonitor monitor) {
 						// delete profile
-						try {
+						SafeRunner.run(() -> {
 							P2TargetUtils.forceCheckTarget(getTarget());
 							P2TargetUtils.deleteProfile(getTarget().getHandle());
-						} catch (CoreException e) {
-							PDEPlugin.log(e);
-						}
+						});
 						getTarget().resolve(monitor);
 						if (monitor.isCanceled()) {
 							return Status.CANCEL_STATUS;