Bug 563155 - JAXWSJavaCompletionProposalComputer requires UI Thread
Audited code, and it turns out UI Thread is not necessary in this class,
so it can run async.
Signed-off-by: Mickael Istria <mistria@redhat.com>
diff --git a/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/plugin.xml b/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/plugin.xml
index e9deba9..a0f7daf 100755
--- a/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/plugin.xml
+++ b/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/plugin.xml
@@ -74,7 +74,8 @@
<javaCompletionProposalComputer
activate="true"
categoryId="org.eclipse.jst.ws.jaxws.ui.jaxwsProposalComputer"
- class="org.eclipse.jst.ws.internal.jaxws.ui.annotations.contentassist.AnnotationCompletionProposalComputer">
+ class="org.eclipse.jst.ws.internal.jaxws.ui.annotations.contentassist.AnnotationCompletionProposalComputer"
+ requiresUIThread="false">
<partition
type="__java_string">
</partition>
diff --git a/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/src/org/eclipse/jst/ws/internal/jaxws/ui/annotations/contentassist/AnnotationCompletionProposalComputer.java b/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/src/org/eclipse/jst/ws/internal/jaxws/ui/annotations/contentassist/AnnotationCompletionProposalComputer.java
index aac307c..2e70cd2 100644
--- a/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/src/org/eclipse/jst/ws/internal/jaxws/ui/annotations/contentassist/AnnotationCompletionProposalComputer.java
+++ b/jaxws/bundles/org.eclipse.jst.ws.jaxws.ui/src/org/eclipse/jst/ws/internal/jaxws/ui/annotations/contentassist/AnnotationCompletionProposalComputer.java
@@ -31,6 +31,7 @@
import org.eclipse.jdt.ui.text.java.IJavaCompletionProposalComputer;
import org.eclipse.jdt.ui.text.java.JavaContentAssistInvocationContext;
import org.eclipse.jface.text.contentassist.ICompletionProposal;
+import org.eclipse.jface.text.contentassist.IContextInformation;
import org.eclipse.jst.ws.annotations.core.AnnotationsManager;
import org.eclipse.jst.ws.annotations.core.initialization.IAnnotationAttributeInitializer;
import org.eclipse.jst.ws.annotations.core.utils.AnnotationUtils;
@@ -42,8 +43,7 @@
super();
}
- @SuppressWarnings("unchecked")
- public List computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) {
+ public List<ICompletionProposal> computeCompletionProposals(ContentAssistInvocationContext context, IProgressMonitor monitor) {
if (context instanceof JavaContentAssistInvocationContext) {
return computeCompletionProposals((JavaContentAssistInvocationContext) context);
}
@@ -151,8 +151,7 @@
return false;
}
- @SuppressWarnings("unchecked")
- public List computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) {
+ public List<IContextInformation> computeContextInformation(ContentAssistInvocationContext context, IProgressMonitor monitor) {
return Collections.emptyList();
}
