*** empty log message ***
diff --git a/bundles/org.eclipse.wst.xsl.core/src/org/eclipse/wst/xsl/core/XSLCore.java b/bundles/org.eclipse.wst.xsl.core/src/org/eclipse/wst/xsl/core/XSLCore.java
index 9c49381..70e19a8 100644
--- a/bundles/org.eclipse.wst.xsl.core/src/org/eclipse/wst/xsl/core/XSLCore.java
+++ b/bundles/org.eclipse.wst.xsl.core/src/org/eclipse/wst/xsl/core/XSLCore.java
@@ -24,6 +24,7 @@
 import org.eclipse.wst.xsl.core.internal.model.Stylesheet;
 import org.eclipse.wst.xsl.core.internal.model.StylesheetBuilder;
 import org.eclipse.wst.xsl.core.internal.model.StylesheetModel;
+import org.eclipse.wst.xsl.core.internal.util.FileUtil;
 
 /**
  * The interface to all aspects of the XSL core functionality.
@@ -126,7 +127,18 @@
 		// TODO depends on how we resolve URIs
 		return currentFile.getParent().getFile(new Path(uri));
 	}
-	
+	/**
+	 * Determine whether the given file is an XML file by inspecting its content types.
+	 * 
+	 * @param file the file to inspect
+	 * @return true if this file is an XML file
+	 */
+
+	public static boolean isXMLFile(IFile file)
+	{
+		return FileUtil.isXMLFile(file);
+	}
+
 	/**
 	 * Determine whether the given file is an XSL file by inspecting its content types.
 	 * 
@@ -135,15 +147,6 @@
 	 */
 	public static boolean isXSLFile(IFile file)
 	{
-		IContentTypeManager contentTypeManager = Platform.getContentTypeManager();
-		IContentType[] types = contentTypeManager.findContentTypesFor(file.getName());
-		for (IContentType contentType : types)
-		{
-			if (contentType.equals(contentTypeManager.getContentType(XSL_CONTENT_TYPE)))
-			{
-				return true;
-			}
-		}
-		return false;
+		return FileUtil.isXSLFile(file);
 	}
 }
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.core.prefs b/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.core.prefs
index 863067b..e7a0088 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.core.prefs
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.core.prefs
@@ -1,4 +1,4 @@
-#Fri Jan 04 01:05:33 EST 2008
+#Fri May 09 15:32:36 BST 2008
 eclipse.preferences.version=1
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
 org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.5
@@ -7,6 +7,19 @@
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
+org.eclipse.jdt.core.compiler.doc.comment.support=enabled
 org.eclipse.jdt.core.compiler.problem.assertIdentifier=error
 org.eclipse.jdt.core.compiler.problem.enumIdentifier=error
+org.eclipse.jdt.core.compiler.problem.invalidJavadoc=warning
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=disabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=disabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=public
+org.eclipse.jdt.core.compiler.problem.missingJavadocComments=warning
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagDescription=return_tag
+org.eclipse.jdt.core.compiler.problem.missingJavadocTags=warning
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsVisibility=public
 org.eclipse.jdt.core.compiler.source=1.5
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.ui.prefs b/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.ui.prefs
new file mode 100644
index 0000000..bf7b476
--- /dev/null
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/.settings/org.eclipse.jdt.ui.prefs
@@ -0,0 +1,3 @@
+#Fri May 09 15:32:34 BST 2008
+eclipse.preferences.version=1
+org.eclipse.jdt.ui.text.custom_code_templates=<?xml version\="1.0" encoding\="UTF-8" standalone\="no"?><templates/>
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/AbstractTableBlock.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/AbstractTableBlock.java
index e121c88..9eb4d8e 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/AbstractTableBlock.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/AbstractTableBlock.java
@@ -14,6 +14,13 @@
 import org.eclipse.jface.dialogs.IDialogSettings;
 import org.eclipse.swt.widgets.Table;
 
+/**
+ * An <code>AbstractLaunchConfigurationTab</code> specialised for blocks that
+ * contain a table. This abstract class conveniently saves and restores the table's
+ * column settings.
+ * 
+ * @author Doug Satchwell
+ */
 public abstract class AbstractTableBlock extends AbstractLaunchConfigurationTab
 {
 	private int fSortColumn;
@@ -45,8 +52,6 @@
 
 	/**
 	 * Restore table settings from the given dialog store using the given key.
-	 * 
-	 * @param widths
 	 */
 	public void restoreColumnSettings()
 	{
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/Messages.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/Messages.java
index f543dc8..147e85d 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/Messages.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/Messages.java
@@ -1,21 +1,47 @@
+/*******************************************************************************
+ * Copyright (c) 2007 Chase Technology Ltd - http://www.chasetechnology.co.uk
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     Doug Satchwell (Chase Technology Ltd) - initial API and implementation
+ *******************************************************************************/
 package org.eclipse.wst.xsl.internal.debug.ui;
 
 import java.util.MissingResourceException;
 import java.util.ResourceBundle;
 
-public class Messages {
+/**
+ * Messages for the debug.ui package.
+ * 
+ * @author Doug Satchwell
+ */
+public class Messages
+{
 	private static final String BUNDLE_NAME = "org.eclipse.wst.xsl.internal.debug.ui.messages"; //$NON-NLS-1$
 
-	private static final ResourceBundle RESOURCE_BUNDLE = ResourceBundle
-			.getBundle(BUNDLE_NAME);
+	private static final ResourceBundle RESOURCE_BUNDLE = ResourceBundle.getBundle(BUNDLE_NAME);
 
-	private Messages() {
+	private Messages()
+	{
 	}
 
-	public static String getString(String key) {
-		try {
+	/**
+	 * Get the message for the given key.
+	 * 
+	 * @param key the message key
+	 * @return the message
+	 */
+	public static String getString(String key)
+	{
+		try
+		{
 			return RESOURCE_BUNDLE.getString(key);
-		} catch (MissingResourceException e) {
+		}
+		catch (MissingResourceException e)
+		{
 			return '!' + key + '!';
 		}
 	}
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/ResourceSelectionBlock.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/ResourceSelectionBlock.java
index a108cc2..ded36e4 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/ResourceSelectionBlock.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/ResourceSelectionBlock.java
@@ -51,6 +51,12 @@
 import org.eclipse.ui.model.WorkbenchLabelProvider;
 import org.eclipse.ui.views.navigator.ResourceComparator;
 
+/**
+ * A block that shows a text box with buttons for browsing workspace or the filesystem
+ * in order to populate the text box with a file path.
+ * 
+ * @author Doug Satchwell
+ */
 public abstract class ResourceSelectionBlock extends AbstractLaunchConfigurationTab
 {
 	protected static final int ERROR_DIRECTORY_NOT_SPECIFIED = 1;
@@ -130,26 +136,55 @@
 		}
 	}
 
+	/**
+	 * Same as <code>new ResourceSelectionBlock(true)</code>
+	 */
 	public ResourceSelectionBlock()
 	{
 		this(true);
 	}
 
+	/**
+	 * Same as <code>new ResourceSelectionBlock(IResource.FOLDER,showDefault)</code>
+	 * 
+	 * @param showDefault true if this should have a 'Show Default' button
+	 */
 	public ResourceSelectionBlock(boolean showDefault)
 	{
 		this(IResource.FOLDER, showDefault);
 	}
 
+	/**
+	 * Same as <code>new ResourceSelectionBlock(resourceType,showDefault,true)</code>
+	 * 
+	 * @param resourceType the type of resource to select - IResource.FOLDER or IResource.FILE
+	 * @param showDefault true if this should have a 'Show Default' button
+	 */
 	public ResourceSelectionBlock(int resourceType, boolean showDefault)
 	{
 		this(resourceType, showDefault, true);
 	}
 
+	/**
+	 * Same as <code>new ResourceSelectionBlock(resourceType,showDefault,required,true)</code>
+	 * 
+	 * @param resourceType the type of resource to select - IResource.FOLDER or IResource.FILE
+	 * @param showDefault true if this should have a 'Show Default' button
+	 * @param required true if a blank text box is invalid
+	 */
 	public ResourceSelectionBlock(int resourceType, boolean showDefault, boolean required)
 	{
 		this(resourceType, showDefault, required, true);
 	}
 
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param resourceType the type of resource to select - IResource.FOLDER or IResource.FILE
+	 * @param showDefault true if this should have a 'Show Default' button
+	 * @param required true if a blank text box is invalid
+	 * @param mustExist true of the selected resource must already exist
+	 */
 	public ResourceSelectionBlock(int resourceType, boolean showDefault, boolean required, boolean mustExist)
 	{
 		super();
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIConstants.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIConstants.java
index 823641d..a77db76 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIConstants.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIConstants.java
@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2007 Chase Technology Ltd - http://www.chasetechnology.co.uk
+ * Copyright (c) 2008 Chase Technology Ltd - http://www.chasetechnology.co.uk
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -10,17 +10,40 @@
  *******************************************************************************/
 package org.eclipse.wst.xsl.internal.debug.ui;
 
-
+/**
+ * Constants for the XSL Debug UI.
+ * 
+ * @author Doug Satchwell
+ */
 public class XSLDebugUIConstants
 {
 	private static final String PREFIX = XSLDebugUIPlugin.PLUGIN_ID + '.';
+	/**
+	 * Constant used to store column setting preferences for the <code>InstalledProcessorsBlock</code>
+	 */
 	public static final String PROCESSOR_DETAILS_DIALOG = PREFIX + "PROCESSOR_DETAILS_DIALOG"; //$NON-NLS-1$
-	public static final String PROCESSOR_OUTPUT_PROPERTIES_BLOCK = PREFIX + "PROCESSOR_OUTPUT_PROPERTIES_BLOCK"; //$NON-NLS-1$
+	/**
+	 * Constant used to store column setting preferences for the <code>ParametersBlock</code>
+	 */
 	public static final String MAIN_PARAMATERS_BLOCK = PREFIX + "MAIN_PARAMATERS_BLOCK"; //$NON-NLS-1$
+	/**
+	 * Constant used to store column setting preferences for the <code>TransformsBlock</code>
+	 */
 	public static final String MAIN_TRANSFORMS_BLOCK = PREFIX + "MAIN_TRANSFORMS_BLOCK"; //$NON-NLS-1$
+	/**
+	 * Constant used to store column setting preferences for the <code>FeaturesBlock</code>
+	 */
 	public static final String FEATURES_BLOCK = PREFIX + "FEATURES_BLOCK"; //$NON-NLS-1$
+	/**
+	 * Constant used to store column setting preferences for the <code>OutputBlock</code>
+	 */
 	public static final String OUTPUT_BLOCK = PREFIX + "OUTPUT_BLOCK"; //$NON-NLS-1$
+	/**
+	 * Constant used to store column setting preferences for the <code>FeaturesBlock</code>
+	 */
 	public static final String FEATURES_LAUNCH_BLOCK = PREFIX + "FEATURES_LAUNCH_BLOCK"; //$NON-NLS-1$
-
+	/**
+	 * The id of the XSL editor
+	 */
 	public static final String XSL_EDITOR_ID = "org.eclipse.wst.xml.ui.internal.tabletree.XMLMultiPageEditorPart"; //$NON-NLS-1$
 }
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIPlugin.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIPlugin.java
index c25f3ce..a946d4e 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIPlugin.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLDebugUIPlugin.java
@@ -1,3 +1,13 @@
+/*******************************************************************************
+ * Copyright (c) 2008 Chase Technology Ltd - http://www.chasetechnology.co.uk
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     Doug Satchwell (Chase Technology Ltd) - initial API and implementation
+ *******************************************************************************/
 package org.eclipse.wst.xsl.internal.debug.ui;
 
 import org.eclipse.core.runtime.CoreException;
@@ -19,17 +29,23 @@
 import org.osgi.framework.BundleContext;
 
 /**
- * The activator class controls the plug-in life cycle
+ * The XSL Debug UI plugin.
+ * 
+ * @author Doug Satchwell
  */
 public class XSLDebugUIPlugin extends AbstractUIPlugin
 {
-
-	// The plug-in ID
+	/**
+	 * The id of this.
+	 */
 	public static final String PLUGIN_ID = "org.eclipse.wst.xsl.debug.ui"; //$NON-NLS-1$
 
 	// The shared instance
 	private static XSLDebugUIPlugin plugin;
 
+	/**
+	 * Create a new instance of this.
+	 */
 	public XSLDebugUIPlugin()
 	{
 		plugin = this;
@@ -48,16 +64,33 @@
 		super.stop(context);
 	}
 
+	/**
+	 * Get the singleton instance of this.
+	 * 
+	 * @return the singleton
+	 */
 	public static XSLDebugUIPlugin getDefault()
 	{
 		return plugin;
 	}
 
+	/**
+	 * Get an ImageDescriptor from a path in this bundle.
+	 * 
+	 * @param path the path to the image
+	 * @return the ImageDescriptor
+	 */
 	public static ImageDescriptor getImageDescriptor(String path)
 	{
 		return imageDescriptorFromPlugin(PLUGIN_ID, path);
 	}
 
+	/**
+	 * Convenience method for opening a given preference page.
+	 * 
+	 * @param id the id of the preference page
+	 * @param page the preference page to show
+	 */
 	public static void showPreferencePage(String id, IPreferencePage page)
 	{
 		final IPreferenceNode targetNode = new PreferenceNode(id, page);
@@ -78,6 +111,11 @@
 		});
 	}
 
+	/**
+	 * Get the current Display if possible, or else the default Display.
+	 * 
+	 * @return the current or default Display
+	 */
 	public static Display getStandardDisplay()
 	{
 		Display display;
@@ -87,11 +125,21 @@
 		return display;
 	}
 
+	/**
+	 * Get the active workbench window from the workbench.
+	 * 
+	 * @return the active workbench window
+	 */
 	public static IWorkbenchWindow getActiveWorkbenchWindow()
 	{
 		return getDefault().getWorkbench().getActiveWorkbenchWindow();
 	}
 
+	/**
+	 * Get the shell of the active workbench window.
+	 * 
+	 * @return the active workbench shell
+	 */
 	public static Shell getActiveWorkbenchShell()
 	{
 		IWorkbenchWindow window = getActiveWorkbenchWindow();
@@ -102,6 +150,11 @@
 		return null;
 	}
 
+	/**
+	 * Get the active page of the active workbench window.
+	 * 
+	 * @return the active page
+	 */
 	public static IWorkbenchPage getActivePage()
 	{
 		IWorkbenchWindow w = getActiveWorkbenchWindow();
@@ -112,11 +165,21 @@
 		return null;
 	}
 
+	/**
+	 * Log the given exception by creating a new Status.
+	 * 
+	 * @param e the exception to log
+	 */
 	public static void log(Exception e)
 	{
 		getDefault().getLog().log(new Status(IStatus.ERROR, PLUGIN_ID, 0, "", e)); //$NON-NLS-1$
 	}
 
+	/**
+	 * Log the given core exception.
+	 * 
+	 * @param e the exception to log
+	 */
 	public static void log(CoreException e)
 	{
 		getDefault().getLog().log(e.getStatus());
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLLaunchConfigurationTab.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLLaunchConfigurationTab.java
index 144a41a..9806a54 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLLaunchConfigurationTab.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLLaunchConfigurationTab.java
@@ -19,6 +19,12 @@
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.Composite;
 
+/**
+ * The base class for XSL launch configuration tabs which allows a number of 'blocks' to be added.
+ * Each block must itself fully implement <code>ILaunchConfigurationTab</code>.
+ * 
+ * @author Doug Satchwell
+ */
 public abstract class XSLLaunchConfigurationTab extends AbstractLaunchConfigurationTab
 {
 	private ILaunchConfigurationTab[] blocks;
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLTTabGroup.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLTTabGroup.java
index 339ec51..49af051 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLTTabGroup.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/XSLTTabGroup.java
@@ -21,8 +21,16 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.output.OutputTab;
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.processor.XSLProcessorTab;
 
+/**
+ *The tab group for the XSL tabs.
+ * 
+ * @author Doug Satchwell
+ */
 public class XSLTTabGroup extends AbstractLaunchConfigurationTabGroup
 {
+	/**
+	 * Create a new instance of this.
+	 */
 	public XSLTTabGroup()
 	{
 	}
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractParameterAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractParameterAction.java
index f1136d8..d19b75b 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractParameterAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractParameterAction.java
@@ -18,13 +18,13 @@
 import org.eclipse.ui.actions.SelectionListenerAction;
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.ParameterViewer;
 
+/**
+ * A convenient base class for add/remove parameter actions.
+ * 
+ * @author Doug Satchwell
+ */
 public abstract class AbstractParameterAction extends SelectionListenerAction
 {
-	public static final int DEFAULT = 0;
-	public static final int ADD = 1;
-	public static final int REMOVE = 2;
-	public static final int MOVE = 3;
-
 	private ParameterViewer viewer;
 	private Button button;
 	private Shell shell;
@@ -35,6 +35,11 @@
 		setViewer(viewer);
 	}
 
+	/**
+	 * Set the viewer.
+	 * 
+	 * @param viewer the parametere viewer
+	 */
 	public void setViewer(ParameterViewer viewer)
 	{
 		if (this.viewer != null)
@@ -54,6 +59,11 @@
 		return viewer;
 	}
 
+	/**
+	 * Set the button associated with the action.
+	 * 
+	 * @param button the button
+	 */
 	public void setButton(Button button)
 	{
 		this.button = button;
@@ -92,6 +102,11 @@
 		return shell;
 	}
 
+	/**
+	 * Set the shell that will be used for opening a dialog.
+	 * 
+	 * @param shell the shell to use
+	 */
 	public void setShell(Shell shell)
 	{
 		this.shell = shell;
@@ -105,6 +120,6 @@
 
 	protected int getActionType()
 	{
-		return DEFAULT;
+		return AbstractStylesheetAction.DEFAULT;
 	}
 }
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractStylesheetAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractStylesheetAction.java
index 1e4a1e9..d636b42 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractStylesheetAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AbstractStylesheetAction.java
@@ -23,11 +23,28 @@
 import org.eclipse.wst.xsl.launching.config.LaunchPipeline;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * A convenient base class for actions associated with the stylesheet viewer.
+ * 
+ * @author Doug Satchwell
+ */
 public abstract class AbstractStylesheetAction extends SelectionListenerAction
 {
+	/**
+	 * The default action type.
+	 */
 	public static final int DEFAULT = 0;
+	/**
+	 * The action type for add actions.
+	 */
 	public static final int ADD = 1;
+	/**
+	 * The action type for remove actions.
+	 */
 	public static final int REMOVE = 2;
+	/**
+	 * The action type for move actions.
+	 */
 	public static final int MOVE = 3;
 
 	private StylesheetViewer viewer;
@@ -40,11 +57,21 @@
 		setViewer(viewer);
 	}
 
+	/**
+	 * Add an array of LaunchTransform's to the viewer.
+	 * 
+	 * @param res launch transforms to add
+	 */
 	public void addTransforms(LaunchTransform[] res)
 	{
 		viewer.addTransforms(res);
 	}
 
+	/**
+	 * Set the viewer associated with this action.
+	 * 
+	 * @param viewer the viewer
+	 */
 	public void setViewer(StylesheetViewer viewer)
 	{
 		if (this.viewer != null)
@@ -64,6 +91,11 @@
 		return viewer;
 	}
 
+	/**
+	 * Set the button associated with this action.
+	 * 
+	 * @param button the button
+	 */
 	public void setButton(Button button)
 	{
 		this.button = button;
@@ -102,6 +134,11 @@
 		return shell;
 	}
 
+	/**
+	 * Set the shell to be used for opening a dialog.
+	 * 
+	 * @param shell the shell to use
+	 */
 	public void setShell(Shell shell)
 	{
 		this.shell = shell;
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ActionMessages.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ActionMessages.java
index 6a2ea22..6b93293 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ActionMessages.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ActionMessages.java
@@ -12,42 +12,88 @@
 
 import org.eclipse.osgi.util.NLS;
 
+/**
+ * Messages for the debug UI actions.
+ * 
+ * @author Doug Satchwell
+ */
 public final class ActionMessages extends NLS
 {
 	private static final String BUNDLE_NAME = "org.eclipse.wst.xsl.internal.debug.ui.actions.ActionMessages"; //$NON-NLS-1$
 
+	/**
+	 * A name for the remove action.
+	 */
 	public static String RemoveAction_Text;
 
+	/**
+	 * A name for the <code>AddWorkspaceFileAction</code> action.
+	 */
 	public static String AddWorkspaceFileAction_Text;
 
+	/**
+	 * A name for the <code>AddExternalFileAction</code> action.
+	 */
 	public static String AddExternalFileAction_Text;
 
+	/**
+	 * A name for the <code>MoveDownAction</code> action.
+	 */
 	public static String MoveDownAction_Text;
 
+	/**
+	 * A name for the <code>MoveUpAction</code> action.
+	 */
 	public static String MoveUpAction_Text;
 
+	/**
+	 * A heading for the <code>AddExternalFileAction</code> dialog.
+	 */
 	public static String AddExternalFileAction_Selection_3;
 
+	/**
+	 * A message for the <code>AddWorkspaceFileAction</code> dialog.
+	 */
 	public static String AddWorkspaceFileAction_DialogMessage;
 
+	/**
+	 * A title for the <code>AddWorkspaceFileAction</code> dialog.
+	 */
 	public static String AddWorkspaceFileAction_DialogTitle;
 
-	public static String EditStylesheetParametersAction_Text;
-
+	/**
+	 * A name for the <code>AddParameterAction</code> action.
+	 */
 	public static String AddParameterAction;
 
+	/**
+	 * A name for the <code>RemoveParameterAction</code> action.
+	 */
 	public static String RemoveParameterAction;
 
+	/**
+	 * A label for the Variables button.
+	 */
 	public static String VariablesFieldButton_Text;
 
+	/**
+	 * A title for the <code>AddParameterAction</code> dialog.
+	 */
 	public static String AddParameterAction_Dialog;
 
-	public static String AddParameterAction_Dialog_Label;
-
+	/**
+	 * A label for the <code>AddParameterAction</code> name text box.
+	 */
 	public static String AddParameterAction_Dialog_Name;
 
+	/**
+	 * A label for the <code>AddParameterAction</code> type combo.
+	 */
 	public static String AddParameterAction_Dialog_Type;
 
+	/**
+	 * A label for the <code>AddParameterAction</code> value text box.
+	 */
 	public static String AddParameterAction_Dialog_Value;
 
 	static
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddExternalFileAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddExternalFileAction.java
index 0c99add..0450d91 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddExternalFileAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddExternalFileAction.java
@@ -17,9 +17,19 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * An action that opens a dialog to allow the user to select a file from the file system.
+ * 
+ * @author Doug Satchwell
+ */
 public class AddExternalFileAction extends OpenDialogAction
 {
-
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer the stylesheet viewer
+	 * @param dialogSettingsPrefix a prefix to use for saving dialog preferences
+	 */
 	public AddExternalFileAction(StylesheetViewer viewer, String dialogSettingsPrefix)
 	{
 		super(ActionMessages.AddExternalFileAction_Text, viewer, dialogSettingsPrefix);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddParameterAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddParameterAction.java
index 163087d..1bafe5d 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddParameterAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddParameterAction.java
@@ -14,12 +14,22 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.ParameterViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchAttribute;
 
+/**
+ * An action that opens a dialog to allow the user to add a parameter to a transform.
+ * 
+ * @author Doug Satchwell
+ */
 public class AddParameterAction extends AbstractParameterAction
 {
 	final String[] types = new String[]
 	{ LaunchAttribute.TYPE_STRING, LaunchAttribute.TYPE_BOOLEAN, LaunchAttribute.TYPE_INT, LaunchAttribute.TYPE_DOUBLE, LaunchAttribute.TYPE_FLOAT, LaunchAttribute.TYPE_OBJECT,
 			LaunchAttribute.TYPE_CLASS, };
 
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer teh viewer
+	 */
 	public AddParameterAction(ParameterViewer viewer)
 	{
 		super(ActionMessages.AddParameterAction, viewer);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddWorkspaceFileAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddWorkspaceFileAction.java
index c1aeae6..ff487df 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddWorkspaceFileAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/AddWorkspaceFileAction.java
@@ -10,12 +10,10 @@
  *******************************************************************************/
 package org.eclipse.wst.xsl.internal.debug.ui.actions;
 
-import org.eclipse.core.resources.IContainer;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IResource;
 import org.eclipse.core.resources.ResourcesPlugin;
 import org.eclipse.core.runtime.IStatus;
-import org.eclipse.core.runtime.Platform;
 import org.eclipse.core.runtime.Status;
 import org.eclipse.jface.viewers.ILabelProvider;
 import org.eclipse.jface.viewers.ITreeContentProvider;
@@ -26,12 +24,17 @@
 import org.eclipse.ui.dialogs.ISelectionStatusValidator;
 import org.eclipse.ui.model.WorkbenchContentProvider;
 import org.eclipse.ui.model.WorkbenchLabelProvider;
-import org.eclipse.ui.views.navigator.ResourceSorter;
-import org.eclipse.wst.xsl.core.internal.util.FileUtil;
+import org.eclipse.ui.views.navigator.ResourceComparator;
+import org.eclipse.wst.xsl.core.XSLCore;
 import org.eclipse.wst.xsl.internal.debug.ui.XSLDebugUIPlugin;
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * An action that opens a dialog to allow the user to select a file in the workspace.
+ * 
+ * @author Doug Satchwell
+ */
 public class AddWorkspaceFileAction extends AbstractStylesheetAction
 {
 	private final ISelectionStatusValidator validator = new ISelectionStatusValidator()
@@ -53,6 +56,11 @@
 		}
 	};
 
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer the viewer
+	 */
 	public AddWorkspaceFileAction(StylesheetViewer viewer)
 	{
 		super(ActionMessages.AddWorkspaceFileAction_Text, viewer);
@@ -81,14 +89,14 @@
 				IResource resource = (IResource)element;
 				if (resource.getType() == IResource.FILE)
 				{
-					if (!FileUtil.isXSLFile((IFile)resource))
+					if (!XSLCore.isXSLFile((IFile)resource))
 						return false;
 				}
 				return true;
 			}
 		});
 		dialog.setInput(ResourcesPlugin.getWorkspace().getRoot());
-		dialog.setSorter(new ResourceSorter(ResourceSorter.NAME));
+		dialog.setComparator(new ResourceComparator(ResourceComparator.NAME));
 
 		if (dialog.open() == Window.OK)
 		{
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ControlAccessibleListener.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ControlAccessibleListener.java
index 423ff7f..71caaf6 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ControlAccessibleListener.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/ControlAccessibleListener.java
@@ -14,11 +14,16 @@
 import org.eclipse.swt.accessibility.AccessibleEvent;
 import org.eclipse.swt.widgets.Control;
 
+/**
+ * Used to help with accessibility.
+ * 
+ * @author Doug Satchwell
+ */
 public class ControlAccessibleListener extends AccessibleAdapter
 {
 	private final String controlName;
 
-	public ControlAccessibleListener(String name)
+	private ControlAccessibleListener(String name)
 	{
 		controlName = name;
 	}
@@ -29,6 +34,12 @@
 		e.result = controlName;
 	}
 
+	/**
+	 * Helper for adding an instance of this to the given control.
+	 * 
+	 * @param comp the control to add this to
+	 * @param name the name for this
+	 */
 	public static void addListener(Control comp, String name)
 	{
 		// strip mnemonic
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveDownAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveDownAction.java
index 22938cf..215619b 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveDownAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveDownAction.java
@@ -16,8 +16,18 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * The action for moving a stylesheet further down in the transformation pipeline.
+ * 
+ * @author Doug Satchwell
+ */
 public class MoveDownAction extends AbstractStylesheetAction
 {
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer
+	 */
 	public MoveDownAction(StylesheetViewer viewer)
 	{
 		super(ActionMessages.MoveDownAction_Text, viewer);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveUpAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveUpAction.java
index 906f529..794548b 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveUpAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MoveUpAction.java
@@ -17,8 +17,18 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * The action for moving a stylesheet further up in the transformation pipeline.
+ * 
+ * @author Doug Satchwell
+ */
 public class MoveUpAction extends AbstractStylesheetAction
 {
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer
+	 */
 	public MoveUpAction(StylesheetViewer viewer)
 	{
 		super(ActionMessages.MoveUpAction_Text, viewer);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MultipleInputDialog.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MultipleInputDialog.java
index f2df4bb..0020ed1 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MultipleInputDialog.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/MultipleInputDialog.java
@@ -37,6 +37,11 @@
 import org.eclipse.swt.widgets.Text;
 import org.eclipse.wst.xsl.internal.debug.ui.XSLDebugUIPlugin;
 
+/**
+ * A dialog with handy methods for adding simple controls to itself.
+ * 
+ * @author Doug Satchwell
+ */
 public class MultipleInputDialog extends Dialog
 {
 	protected static final String FIELD_NAME = "FIELD_NAME"; //$NON-NLS-1$
@@ -50,11 +55,18 @@
 	protected List<FieldSummary> fieldList = new ArrayList<FieldSummary>();
 	protected List<Scrollable> controlList = new ArrayList<Scrollable>();
 	protected List<Validator> validators = new ArrayList<Validator>();
+	@SuppressWarnings("unchecked")
 	protected Map<Object, Comparable> valueMap = new HashMap<Object, Comparable>();
 
 	private final String title;
 
-	public MultipleInputDialog(Shell shell, String title)
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param shell the shell to open the dialog on
+	 * @param title the title for the dialog
+	 */
+	public MultipleInputDialog(final Shell shell, final String title)
 	{
 		super(shell);
 		this.title = title;
@@ -125,21 +137,49 @@
 		return container;
 	}
 
+	/**
+	 * Add a label, text box and button for browsing the for a file.
+	 * 
+	 * @param labelText the label
+	 * @param initialValue the initial value
+	 * @param allowsEmpty true if the text box can be empty
+	 */
 	public void addBrowseField(String labelText, String initialValue, boolean allowsEmpty)
 	{
 		fieldList.add(new FieldSummary(BROWSE, labelText, initialValue, allowsEmpty));
 	}
 
+	/**
+	 * Add a label and a text box.
+	 * 
+	 * @param labelText the label
+	 * @param initialValue the initial value
+	 * @param allowsEmpty true if the text box can be empty
+	 */
 	public void addTextField(String labelText, String initialValue, boolean allowsEmpty)
 	{
 		fieldList.add(new FieldSummary(TEXT, labelText, initialValue, allowsEmpty));
 	}
 
+	/**
+	 * Add a label, a text box and a button for selecting variables.
+	 * 
+	 * @param labelText the label
+	 * @param initialValue the initial value
+	 * @param allowsEmpty true if the text box can be empty
+	 */
 	public void addVariablesField(String labelText, String initialValue, boolean allowsEmpty)
 	{
 		fieldList.add(new FieldSummary(VARIABLE, labelText, initialValue, allowsEmpty));
 	}
 
+	/**
+	 * Add a label and a combo.
+	 * 
+	 * @param labelText the label
+	 * @param initialIndex the initial selection index
+	 * @param items the array of items for the combo
+	 */
 	public void addComboField(String labelText, int initialIndex, String[] items)
 	{
 		fieldList.add(new FieldSummary(COMBO, labelText, items, initialIndex));
@@ -185,7 +225,7 @@
 		controlList.add(text);
 	}
 
-	public void createVariablesField(String labelText, String initialValue, boolean allowEmpty)
+	protected void createVariablesField(String labelText, String initialValue, boolean allowEmpty)
 	{
 		Label label = new Label(panel, SWT.NONE);
 		label.setText(labelText);
@@ -255,7 +295,7 @@
 
 	}
 
-	public void createComboField(String labelText, int initialValue, String[] items)
+	protected void createComboField(String labelText, int initialValue, String[] items)
 	{
 		Label label = new Label(panel, SWT.NONE);
 		label.setText(labelText);
@@ -311,22 +351,22 @@
 		return super.open();
 	}
 
-	public Object getValue(String key)
+	protected Object getValue(String key)
 	{
 		return valueMap.get(key);
 	}
 
-	public String getStringValue(String key)
+	protected String getStringValue(String key)
 	{
 		return (String) getValue(key);
 	}
 
-	public int getIntValue(String key)
+	protected int getIntValue(String key)
 	{
 		return ((Integer) getValue(key)).intValue();
 	}
 
-	public void validateFields()
+	protected void validateFields()
 	{
 		for (Iterator<Validator> i = validators.iterator(); i.hasNext();)
 		{
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/OpenDialogAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/OpenDialogAction.java
index b8de6ee..3c269b0 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/OpenDialogAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/OpenDialogAction.java
@@ -14,11 +14,23 @@
 import org.eclipse.wst.xsl.internal.debug.ui.XSLDebugUIPlugin;
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 
+/**
+ * An action that opens a dialog.
+ * 
+ * @author Doug Satchwell
+ */
 public class OpenDialogAction extends AbstractStylesheetAction
 {
 	protected static final String LAST_PATH_SETTING = "LAST_PATH_SETTING"; //$NON-NLS-1$
 	private String fPrefix = null;
 
+	/**
+	 * Create a new instance of this
+	 * 
+	 * @param label the dialog title
+	 * @param viewer a viewer that this dialog is associated with
+	 * @param dialogSettingsPrefix the prefix to use for saving dialog preferences
+	 */
 	public OpenDialogAction(String label, StylesheetViewer viewer, String dialogSettingsPrefix)
 	{
 		super(label, viewer);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveAction.java
index 9e82c8c..b72ccfe 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveAction.java
@@ -14,13 +14,24 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.StylesheetViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchTransform;
 
+/**
+ * An action that removes a selection from a viewer.
+ * 
+ * @author Doug Satchwell
+ */
 public class RemoveAction extends AbstractStylesheetAction
 {
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer the viewer that will have its selection removed
+	 */
 	public RemoveAction(StylesheetViewer viewer)
 	{
 		super(ActionMessages.RemoveAction_Text, viewer);
 	}
 
+	@SuppressWarnings("unchecked")
 	@Override
 	public void run()
 	{
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveParameterAction.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveParameterAction.java
index d091ca2..124f203 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveParameterAction.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/actions/RemoveParameterAction.java
@@ -16,9 +16,18 @@
 import org.eclipse.wst.xsl.internal.debug.ui.tabs.main.ParameterViewer;
 import org.eclipse.wst.xsl.launching.config.LaunchAttribute;
 
+/**
+ * An action that removes a selection from a viewer.
+ * 
+ * @author Doug Satchwell
+ */
 public class RemoveParameterAction extends AbstractParameterAction
 {
-
+	/**
+	 * Create a new instance of this.
+	 * 
+	 * @param viewer the viewer that will have its selection removed
+	 */
 	public RemoveParameterAction(ParameterViewer viewer)
 	{
 		super(ActionMessages.RemoveParameterAction, viewer);
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/breakpoint/XSLBreakpointProvider.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/breakpoint/XSLBreakpointProvider.java
index 8bab02b..a92735c 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/breakpoint/XSLBreakpointProvider.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/breakpoint/XSLBreakpointProvider.java
@@ -34,6 +34,11 @@
 import org.eclipse.wst.xsl.internal.debug.ui.XSLDebugUIPlugin;
 import org.eclipse.wst.xsl.launching.model.XSLLineBreakpoint;
 
+/**
+ * An SSE <code>IBreakpointProvider</code> for XSL files.
+ * 
+ * @author Doug Satchwell
+ */
 public class XSLBreakpointProvider implements IBreakpointProvider, IExecutableExtension
 {
 	public IStatus addBreakpoint(IDocument document, IEditorInput input, int editorLineNumber, int offset) throws CoreException
@@ -128,7 +133,7 @@
 
 					for (int i = 0; i < partitions.length; ++i)
 					{
-						String type = partitions[i].getType();
+						// String type = partitions[i].getType();
 						result = partitions[i].getOffset();
 					}
 				}
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLLineBreakpointAdapter.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLLineBreakpointAdapter.java
index 90b3340..870045d 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLLineBreakpointAdapter.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLLineBreakpointAdapter.java
@@ -10,6 +10,7 @@
  *******************************************************************************/
 package org.eclipse.wst.xsl.internal.debug.ui.model;
 
+import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IResource;
 import org.eclipse.core.runtime.CoreException;
 import org.eclipse.debug.core.DebugPlugin;
@@ -20,9 +21,15 @@
 import org.eclipse.jface.viewers.ISelection;
 import org.eclipse.ui.IWorkbenchPart;
 import org.eclipse.ui.texteditor.ITextEditor;
+import org.eclipse.wst.xsl.core.XSLCore;
 import org.eclipse.wst.xsl.launching.model.IXSLConstants;
 import org.eclipse.wst.xsl.launching.model.XSLLineBreakpoint;
 
+/**
+ * An <code>IToggleBreakpointsTarget</code> for XSL breakpoints.
+ * 
+ * @author Doug Satchwell
+ */
 public class XSLLineBreakpointAdapter implements IToggleBreakpointsTarget
 {
 	public void toggleLineBreakpoints(IWorkbenchPart part, ISelection selection) throws CoreException
@@ -67,11 +74,9 @@
 		{
 			ITextEditor editorPart = (ITextEditor) part;
 			IResource resource = (IResource) editorPart.getEditorInput().getAdapter(IResource.class);
-			if (resource != null)
+			if (resource != null && resource.getType() == IResource.FILE)
 			{
-				String extension = resource.getFileExtension();
-				// TODO handle other file extensions/content types
-				if (extension != null && extension.equalsIgnoreCase("xsl")) //$NON-NLS-1$
+				if (XSLCore.isXSLFile((IFile)resource))
 				{
 					return editorPart;
 				}
diff --git a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLModelPresentation.java b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLModelPresentation.java
index 01a1631..50ded41 100644
--- a/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLModelPresentation.java
+++ b/bundles/org.eclipse.wst.xsl.debug.ui/src/org/eclipse/wst/xsl/internal/debug/ui/model/XSLModelPresentation.java
@@ -24,6 +24,11 @@
 import org.eclipse.wst.xsl.internal.debug.ui.XSLDebugUIPlugin;
 import org.eclipse.wst.xsl.launching.model.XSLVariable;
 
+/**
+ * An <code>IDebugModelPresentation</code> for XSL debugging. 
+ * 
+ * @author Doug Satchwell
+ */
 public class XSLModelPresentation extends LabelProvider implements IDebugModelPresentation
 {
 	private Image localImg;