Bug 66375 [Dialogs] TaskPropertiesDialog and DialogMarkerProperties need a common superclass
RC1 Despite these two classes being pretty much identical they share almost no code. TaskPropertiesDialog should be a subclass a DialogMarkerProperties. As this is an API change post 3.0
bundles/org.eclipse.ui.ide/src/org/eclipse/ui/views/bookmarkexplorer/BookmarkPropertiesDialog.java
* Saves the changes made in the dialog if needed.
     * Creates a new bookmark if needed.
     * Updates the existing bookmark only if there have been changes.
     */
    private void saveChanges() {
        try {
            ResourcesPlugin.getWorkspace().run(new IWorkspaceRunnable() {
                public void run(IProgressMonitor monitor) throws CoreException {
                    if (marker == null) {
						createMarker();
					} else if (isDirty()) {
						updateMarker();
					}
                }
            }, null);
        } catch (CoreException e) {
            ErrorDialog.openError(getShell(), BookmarkMessages.Error_text, null, e.getStatus());
            return;
        }
    }
    /**
     * Creates or updates the marker.  Must be called within a workspace runnable.
     */
    private void updateMarker() throws CoreException {
        
        
        
        Map attrs = getMarkerAttributesFromDialog();
        for (Iterator i = attrs.keySet().iterator(); i.hasNext();) {
            String key = (String) i.next();