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/NavigatorFilterService.java
activeFilters.add(activeFilterIds[i]);
				}
			} else {
				ICommonFilterDescriptor[] visibleFilterDescriptors = getVisibleFilterDescriptors();
				for (int i = 0; i < visibleFilterDescriptors.length; i++) {
					if (visibleFilterDescriptors[i].isActiveByDefault()) {
						activeFilters.add(visibleFilterDescriptors[i].getId());
					}
				}
			}
		} catch (RuntimeException e) {
			NavigatorPlugin.logError(0, e.getMessage(), e);
		}
	}
	/*
	 * (non-Javadoc)
	 * 
	 * @see org.eclipse.ui.navigator.INavigatorFilterService#persistFilterActivationState()
	 */
	public void persistFilterActivationState() {
		synchronized (activeFilters) {
			/*
			 * by creating a StringBuffer with DELIM, we ensure the string is
			 * not empty when persisted.
			 */
			StringBuffer activatedFiltersPreferenceValue = new StringBuffer(DELIM);
			for (Iterator activeItr = activeFilters.iterator(); activeItr.hasNext();) {
				activatedFiltersPreferenceValue.append(activeItr.next().toString()).append(DELIM);
			}
			IEclipsePreferences prefs = NavigatorContentService.getPreferencesRoot();