[116812] [api] wst.sse APIs need javadoc
diff --git a/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/internal/editor/EditorExecutionContext.java b/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/EditorExecutionContext.java
similarity index 94%
rename from bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/internal/editor/EditorExecutionContext.java
rename to bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/EditorExecutionContext.java
index e487707..388a27d 100644
--- a/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/internal/editor/EditorExecutionContext.java
+++ b/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/EditorExecutionContext.java
@@ -10,17 +10,16 @@
  *     Jens Lukowski/Innoopract - initial renaming/restructuring
  *     
  *******************************************************************************/
-package org.eclipse.wst.sse.ui.internal.editor;
+package org.eclipse.wst.sse.ui;
 
 import org.eclipse.swt.widgets.Display;
 import org.eclipse.ui.IWorkbench;
 import org.eclipse.ui.PlatformUI;
 import org.eclipse.wst.sse.core.internal.IExecutionDelegate;
-import org.eclipse.wst.sse.ui.StructuredTextEditor;
 
 
 
-public class EditorExecutionContext implements IExecutionDelegate {
+class EditorExecutionContext implements IExecutionDelegate {
 
 
 	StructuredTextEditor fEditor;
diff --git a/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/StructuredTextEditor.java b/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/StructuredTextEditor.java
index f9fd72f..e01df97 100644
--- a/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/StructuredTextEditor.java
+++ b/bundles/org.eclipse.wst.sse.ui/src/org/eclipse/wst/sse/ui/StructuredTextEditor.java
@@ -153,7 +153,6 @@
 import org.eclipse.wst.sse.ui.internal.debug.ManageBreakpointAction;
 import org.eclipse.wst.sse.ui.internal.debug.ToggleBreakpointAction;
 import org.eclipse.wst.sse.ui.internal.debug.ToggleBreakpointsTarget;
-import org.eclipse.wst.sse.ui.internal.editor.EditorExecutionContext;
 import org.eclipse.wst.sse.ui.internal.editor.EditorModelUtil;
 import org.eclipse.wst.sse.ui.internal.editor.IHelpContextIds;
 import org.eclipse.wst.sse.ui.internal.editor.StructuredModelDocumentProvider;
@@ -1012,12 +1011,12 @@
 
 
 	/**
-	 * Starts a background operation.
+	 * Starts background mode.
 	 * <p>
 	 * Not API. May be removed in the future.
 	 * </p>
 	 */
-	public void beginBackgroundOperation() {
+	void beginBackgroundOperation() {
 		fBackgroundJobEnded = false;
 		// if already in busy state, no need to do anything
 		// and, we only start, or reset, the timed busy
@@ -1765,13 +1764,12 @@
 	}
 
 	/**
-	 * This is the public method to be called to notifiy us that document is
-	 * being updated by backround job.
+	 * End background mode.
 	 * <p>
 	 * Not API. May be removed in the future.
 	 * </p>
 	 */
-	public void endBackgroundOperation() {
+	void endBackgroundOperation() {
 		fBackgroundJobEnded = true;
 		// note, we don't immediately end our 'internal busy' state,
 		// since we may get many calls in a short period of
@@ -2010,6 +2008,8 @@
 					fisReleased = false;
 				}
 			}
+			
+			// ISSUE: this looks bad ... edit-time factories not initialized unless someone calls getModel? 
 			// factories will not be re-added if already exists
 			EditorModelUtil.addFactoriesTo(fStructuredModel);
 
@@ -2751,7 +2751,7 @@
 		}
 	}
 
-	protected void updateMenuText() {
+	private void updateMenuText() {
 		if (fStructuredModel != null && !fStructuredModel.isModelStateChanging() && getTextViewer().getTextWidget() != null) {
 			// performance: don't force an update of the action bars unless
 			// required as it is expensive