Bug 139986 [CommonNavigator] Convert catch(Error/Throwable) to SafeRunner
In several places the CN framework uses catch(Error/Throwable) to ensure that code does not bring the framework down. Change these to use SafeRunner.
bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/filters/CommonFilterDescriptor.java
NavigatorPlugin
							.log(
									IStatus.WARNING,
									0,
									"A \"commonFilter\" was specified in " + 
											element.getDeclaringExtension().getNamespaceIdentifier()
											+ " which specifies a \"class\" attribute and an Core Expression.\n" + 
											"Only the Core Expression will be respected.", 
									null);
				} 
				return new CoreExpressionFilter(filterExpression);
			}
			return (ViewerFilter) element.createExecutableExtension(ATT_CLASS);
		} catch (RuntimeException re) {
			NavigatorPlugin.logError(0, re.getMessage(), re);
		} catch (CoreException e) {
			NavigatorPlugin.logError(0, e.getMessage(), e);
		}
		return SkeletonViewerFilter.INSTANCE;
	}
	/*
	 * (non-Javadoc)
	 * 
	 * @see java.lang.Object#toString()
	 */
	public String toString() {
		return "CommonFilterDescriptor[" + getName() + " (" + getId() + ")]"; 
	}
}