Author: jm
Date: 2012-03-14 13:15:08 -0700 (Wed, 14 Mar 2012)
New Revision: 28537

Modified:
   
core3/impl/trunk/work-swing-impl/impl/src/main/java/org/cytoscape/work/internal/task/JDialogTaskManager.java
Log:
Fixes #723: Fixed potential deadlock in JDialogTaskManager because UI code 
isn't being run in UI thread

Modified: 
core3/impl/trunk/work-swing-impl/impl/src/main/java/org/cytoscape/work/internal/task/JDialogTaskManager.java
===================================================================
--- 
core3/impl/trunk/work-swing-impl/impl/src/main/java/org/cytoscape/work/internal/task/JDialogTaskManager.java
        2012-03-14 20:03:10 UTC (rev 28536)
+++ 
core3/impl/trunk/work-swing-impl/impl/src/main/java/org/cytoscape/work/internal/task/JDialogTaskManager.java
        2012-03-14 20:15:08 UTC (rev 28537)
@@ -10,6 +10,7 @@
 import java.util.concurrent.TimeUnit;
 
 import javax.swing.JDialog;
+import javax.swing.SwingUtilities;
 
 import org.cytoscape.work.AbstractTaskManager;
 import org.cytoscape.work.Task;
@@ -183,7 +184,12 @@
                        public void run() {
                                if (!(executorFuture.isDone() || 
executorFuture.isCancelled())) {
                                        taskMonitor.setFuture(executorFuture);
-                                       taskMonitor.open();
+                                       SwingUtilities.invokeLater(new 
Runnable() {
+                                               @Override
+                                               public void run() {
+                                                       taskMonitor.open();
+                                               }
+                                       });
                                }
                        }
                };

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to