OpenParentToolbarAction.selectionChanged() can lead to UI freeze

[briandealwis]

I wonder if it's worth instrumenting JavaModelHelper methods to prevent their use from the UI thread.

!SUBENTRY 1 org.eclipse.ui.monitoring 1 0 2017-08-23 12:10:28.631
!MESSAGE Sample at 12:10:28.493 (+0.336s)
Thread 'main' tid=1 (RUNNABLE)
!STACK 0
Stack Trace
	at java.util.zip.ZipFile.open(Native Method)
	at java.util.zip.ZipFile.<init>(ZipFile.java:225)
	at java.util.zip.ZipFile.<init>(ZipFile.java:155)
	at java.util.zip.ZipFile.<init>(ZipFile.java:169)
	at org.eclipse.jdt.internal.core.JavaModelManager.getZipFile(JavaModelManager.java:2860)
	at org.eclipse.jdt.internal.core.JavaModelManager.getZipFile(JavaModelManager.java:2829)
	at org.eclipse.jdt.internal.core.search.matching.JavaSearchNameEnvironment.mapToClassPathLocation(JavaSearchNameEnvironment.java:113)
	at org.eclipse.jdt.internal.core.search.matching.JavaSearchNameEnvironment.computeClasspathLocations(JavaSearchNameEnvironment.java:101)
	at org.eclipse.jdt.internal.core.search.matching.JavaSearchNameEnvironment.<init>(JavaSearchNameEnvironment.java:54)
	at org.eclipse.jdt.internal.core.search.matching.IndexBasedJavaSearchEnvironment.create(IndexBasedJavaSearchEnvironment.java:325)
	at org.eclipse.jdt.internal.core.search.matching.MatchLocator.initialize(MatchLocator.java:1192)
	at org.eclipse.jdt.internal.core.search.matching.MatchLocator.locateMatches(MatchLocator.java:1222)
	at org.eclipse.jdt.internal.core.search.matching.MatchLocator.locateMatches(MatchLocator.java:1338)
	at org.eclipse.jdt.internal.core.search.matching.MatchLocator.locateMatches(MatchLocator.java:1470)
	at org.eclipse.jdt.internal.core.search.JavaSearchParticipant.locateMatches(JavaSearchParticipant.java:122)
	at org.eclipse.jdt.internal.core.search.BasicSearchEngine.findMatches(BasicSearchEngine.java:236)
	at org.eclipse.jdt.internal.core.search.BasicSearchEngine.search(BasicSearchEngine.java:583)
	at org.eclipse.jdt.core.search.SearchEngine.search(SearchEngine.java:600)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.performSearch(JavaModelHelper.java:397)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper$1.call(JavaModelHelper.java:685)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper$1.call(JavaModelHelper.java:1)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.performCreation(JavaModelHelper.java:310)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.resolveOperation(JavaModelHelper.java:256)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.basicResolveType(JavaModelHelper.java:677)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.resolveType(JavaModelHelper.java:669)
	at ca.ubc.cs.ferret.jdt.JavaModelHelper.getSuperclass(JavaModelHelper.java:490)
	at ca.ubc.cs.ferret.jdt.JdtSphereHelper.getParent(JdtSphereHelper.java:266)
	at ca.ubc.cs.ferret.FerretPlugin.getParent(FerretPlugin.java:603)
	at ca.ubc.cs.ferret.views.QueriesDossierView$OpenParentToolbarAction.selectionChanged(QueriesDossierView.java:1467)
	at ca.ubc.cs.ferret.views.QueriesDossierView.setInput(QueriesDossierView.java:1206)
	at ca.ubc.cs.ferret.views.QueriesDossierView.basicPerformQuery(QueriesDossierView.java:1176)
	at ca.ubc.cs.ferret.views.QueriesDossierView.performQuery(QueriesDossierView.java:1138)
	at ca.ubc.cs.ferret.views.QueriesDossierView.handleSelectionChanged(QueriesDossierView.java:546)
	at ca.ubc.cs.ferret.views.QueriesDossierView$7.run(QueriesDossierView.java:525)
	at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:37)
	at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:182)
	at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4046)
	at org.eclipse.swt.widgets.Display.observerProc(Display.java:3626)
	at org.eclipse.swt.internal.cocoa.OS.objc_msgSendSuper(Native Method)
	at org.eclipse.swt.widgets.Display.applicationNextEventMatchingMask(Display.java:5161)
	at org.eclipse.swt.widgets.Display.applicationProc(Display.java:5542)
	at org.eclipse.swt.internal.cocoa.OS.objc_msgSendSuper(Native Method)
	at org.eclipse.swt.widgets.Widget.callSuper(Widget.java:232)
	at org.eclipse.swt.widgets.Widget.mouseDownSuper(Widget.java:1140)
	at org.eclipse.swt.widgets.Tree.mouseDownSuper(Tree.java:2144)
	at org.eclipse.swt.widgets.Widget.mouseDown(Widget.java:1132)
	at org.eclipse.swt.widgets.Control.mouseDown(Control.java:2618)
	at org.eclipse.swt.widgets.Tree.mouseDown(Tree.java:2111)
	at org.eclipse.swt.widgets.Display.windowProc(Display.java:5810)
	at org.eclipse.swt.internal.cocoa.OS.objc_msgSendSuper(Native Method)
	at org.eclipse.swt.widgets.Widget.callSuper(Widget.java:232)
	at org.eclipse.swt.widgets.Widget.windowSendEvent(Widget.java:2152)
	at org.eclipse.swt.widgets.Shell.windowSendEvent(Shell.java:2411)
	at org.eclipse.swt.widgets.Display.windowProc(Display.java:5874)
	at org.eclipse.swt.internal.cocoa.OS.objc_msgSendSuper(Native Method)
	at org.eclipse.swt.widgets.Display.applicationSendEvent(Display.java:5303)
	at org.eclipse.swt.widgets.Display.applicationProc(Display.java:5452)
	at org.eclipse.swt.internal.cocoa.OS.objc_msgSend(Native Method)
	at org.eclipse.swt.internal.cocoa.NSApplication.sendEvent(NSApplication.java:128)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3708)
	at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$5.run(PartRenderingEngine.java:1150)
	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:336)
	at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1039)
	at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:153)
	at org.eclipse.ui.internal.Workbench.lambda$3(Workbench.java:680)
	at org.eclipse.ui.internal.Workbench$$Lambda$15/114565630.run(Unknown Source)
	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:336)
	at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:594)
	at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:148)
	at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:151)
	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:388)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:243)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:653)
	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:590)
	at org.eclipse.equinox.launcher.Main.run(Main.java:1499)