[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