From 4e1da6b73e75a11d0ce19d505b9b0413d85bcfaa Mon Sep 17 00:00:00 2001
From: Eamonn Saunders <esaunders@basistech.com>
Date: Mon, 1 Dec 2014 10:21:57 -0500
Subject: [PATCH] Code that results in UI interaction needs to be invoked on
 the EDT.

---
 .../sleuthkit/autopsy/casemodule/Case.java    | 76 +++++++++++--------
 1 file changed, 46 insertions(+), 30 deletions(-)

diff --git a/Core/src/org/sleuthkit/autopsy/casemodule/Case.java b/Core/src/org/sleuthkit/autopsy/casemodule/Case.java
index 6e833bbacb..ba6360fa00 100644
--- a/Core/src/org/sleuthkit/autopsy/casemodule/Case.java
+++ b/Core/src/org/sleuthkit/autopsy/casemodule/Case.java
@@ -18,6 +18,7 @@
  */
 package org.sleuthkit.autopsy.casemodule;
 
+import java.awt.EventQueue;
 import java.awt.Frame;
 import java.beans.PropertyChangeListener;
 import java.beans.PropertyChangeSupport;
@@ -1077,7 +1078,7 @@ public class Case implements SleuthkitCase.ErrorObserver {
     }
 
     //case change helper
-    private static void doCaseChange(Case toChangeTo) {
+    private static void doCaseChange(final Case toChangeTo) {
         logger.log(Level.INFO, "Changing Case to: " + toChangeTo); //NON-NLS
         if (toChangeTo != null) { // new case is open
 
@@ -1085,39 +1086,49 @@ public class Case implements SleuthkitCase.ErrorObserver {
             Case.clearTempFolder();
             checkSubFolders(toChangeTo);
 
-            // enable these menus
-            CallableSystemAction.get(AddImageAction.class).setEnabled(true);
-            CallableSystemAction.get(CaseCloseAction.class).setEnabled(true);
-            CallableSystemAction.get(CasePropertiesAction.class).setEnabled(true);
-            CallableSystemAction.get(CaseDeleteAction.class).setEnabled(true); // Delete Case menu
+            EventQueue.invokeLater(new Runnable() {
+                @Override
+                public void run() {
+                    // enable these menus
+                    CallableSystemAction.get(AddImageAction.class).setEnabled(true);
+                    CallableSystemAction.get(CaseCloseAction.class).setEnabled(true);
+                    CallableSystemAction.get(CasePropertiesAction.class).setEnabled(true);
+                    CallableSystemAction.get(CaseDeleteAction.class).setEnabled(true); // Delete Case menu
+
+                    if (toChangeTo.hasData()) {
+                        // open all top components
+                        CoreComponentControl.openCoreWindows();
+                    } else {
+                        // close all top components
+                        CoreComponentControl.closeCoreWindows();
+                    }
+                }
+            });
 
-            if (toChangeTo.hasData()) {
-                // open all top components
-                CoreComponentControl.openCoreWindows();
-            } else {
-                // close all top components
-                CoreComponentControl.closeCoreWindows();
-            }
         } else { // case is closed
-            // close all top components first
-            CoreComponentControl.closeCoreWindows();
+            EventQueue.invokeLater(new Runnable() {
+                @Override
+                public void run() {
+                    // close all top components first
+                    CoreComponentControl.closeCoreWindows();
 
-            // disable these menus
-            CallableSystemAction.get(AddImageAction.class).setEnabled(false); // Add Image menu
-            CallableSystemAction.get(CaseCloseAction.class).setEnabled(false); // Case Close menu
-            CallableSystemAction.get(CasePropertiesAction.class).setEnabled(false); // Case Properties menu
-            CallableSystemAction.get(CaseDeleteAction.class).setEnabled(false); // Delete Case menu
+                    // disable these menus
+                    CallableSystemAction.get(AddImageAction.class).setEnabled(false); // Add Image menu
+                    CallableSystemAction.get(CaseCloseAction.class).setEnabled(false); // Case Close menu
+                    CallableSystemAction.get(CasePropertiesAction.class).setEnabled(false); // Case Properties menu
+                    CallableSystemAction.get(CaseDeleteAction.class).setEnabled(false); // Delete Case menu
 
-            //clear pending notifications
-            MessageNotifyUtil.Notify.clear();
+                    //clear pending notifications
+                    MessageNotifyUtil.Notify.clear();
 
+                    Frame f = WindowManager.getDefault().getMainWindow();
+                    f.setTitle(Case.getAppName()); // set the window name to just application name
 
-            Frame f = WindowManager.getDefault().getMainWindow();
-            f.setTitle(Case.getAppName()); // set the window name to just application name
-
-            //try to force gc to happen
-            System.gc();
-            System.gc();
+                    //try to force gc to happen
+                    System.gc();
+                    System.gc();
+                }
+            });
         }
 
         //log memory usage after case changed
@@ -1130,8 +1141,13 @@ public class Case implements SleuthkitCase.ErrorObserver {
     private static void doCaseNameChange(String newCaseName) {
         // update case name
         if (!newCaseName.equals("")) {
-            Frame f = WindowManager.getDefault().getMainWindow();
-            f.setTitle(newCaseName + " - " + Case.getAppName()); // set the window name to the new value
+            EventQueue.invokeLater(new Runnable() {
+                @Override
+                public void run() {
+                    Frame f = WindowManager.getDefault().getMainWindow();
+                    f.setTitle(newCaseName + " - " + Case.getAppName()); // set the window name to the new value
+                }
+            });
         }
     }