/hg/icedtea6: Force Event Dispatch to always get created before ...

[mwong at icedtea.classpath.org]

changeset 4bd31caf5e07 in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=4bd31caf5e07
author: Man Lung Wong <mwong at redhat.com>
date: Fri Jul 30 15:42:22 2010 -0400

	Force Event Dispatch to always get created before so its
	contextclassloader will be set properly. Also made a minor update to
	reflect related bug number to a previous commit made.


diffstat:

3 files changed, 20 insertions(+), 36 deletions(-)
ChangeLog                                                  |   13 ++++
netx/net/sourceforge/jnlp/Launcher.java                    |    7 ++
netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java |   36 ------------

diffs (106 lines):

diff -r 775503860dc5 -r 4bd31caf5e07 ChangeLog
--- a/ChangeLog	Fri Jul 30 12:18:43 2010 +0200
+++ b/ChangeLog	Fri Jul 30 15:42:22 2010 -0400
@@ -1,3 +1,15 @@ 2010-07-29  Omair Majid  <omajid at redhat.
+2010-07-30  Man Lung Wong  <mwong at redhat.com>
+	* netx/net/sourceforge/jnlp/Launcher.java
+	(launchApplication): Make sure an Event Dispatch Thread
+	is created before contextclassloader gets set.
+	* netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java
+	(ContextUpdater): Removed it as it is no longer useful after
+	the above change.
+	(checkTopLevelWindow): Removed the line that adds
+	ContextUpdater as it no longer exists.
+	* NEWS: This fixes one of the problems launching the link
+	mentioned in icedtea bug 486.
+
 2010-07-29  Omair Majid  <omajid at redhat.com>
 
 	* NEWS: Add this fix.
@@ -562,6 +574,7 @@ 2010-07-08  Man Lung Wong  <mwong at redhat
 	be parsed with the encoding the jnlp file is in.
 	(getEncoding): A new method which checks the first four bytes of input
 	and determines what the files encoding is.
+	* NEWS: Fixes icedtea bug 486.
 
 2010-07-08  Andrew John Hughes  <ahughes at redhat.com>
 
diff -r 775503860dc5 -r 4bd31caf5e07 netx/net/sourceforge/jnlp/Launcher.java
--- a/netx/net/sourceforge/jnlp/Launcher.java	Fri Jul 30 12:18:43 2010 +0200
+++ b/netx/net/sourceforge/jnlp/Launcher.java	Fri Jul 30 15:42:22 2010 -0400
@@ -41,6 +41,8 @@ import net.sourceforge.jnlp.services.Ins
 import net.sourceforge.jnlp.services.InstanceExistsException;
 import net.sourceforge.jnlp.services.ServiceUtil;
 import net.sourceforge.jnlp.util.Reflect;
+
+import javax.swing.SwingUtilities;
 
 /**
  * Launches JNLPFiles either in the foreground or background.<p>
@@ -441,6 +443,11 @@ public class Launcher {
 
             Method main = mainClass.getDeclaredMethod("main", new Class[] {String[].class} );
             String args[] = file.getApplication().getArguments();
+            
+            SwingUtilities.invokeAndWait(new Runnable() {
+                // dummy method to force Event Dispatch Thread creation
+                public void run(){}
+            });
 
             setContextClassLoaderForAllThreads(app.getThreadGroup(), app.getClassLoader());
 
diff -r 775503860dc5 -r 4bd31caf5e07 netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java	Fri Jul 30 12:18:43 2010 +0200
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java	Fri Jul 30 15:42:22 2010 -0400
@@ -104,42 +104,8 @@ class JNLPSecurityManager extends Securi
     /** weak reference to most app who's windows was most recently activated */
     private WeakReference activeApplication = null;
 
-    /** listener installs the app's classloader on the event dispatch thread */
-    private ContextUpdater contextListener = new ContextUpdater();
-
     /** Sets whether or not exit is allowed (in the context of the plugin, this is always false) */
     private boolean exitAllowed = true;
-
-    private class ContextUpdater extends WindowAdapter implements PrivilegedAction {
-        private ApplicationInstance app = null;
-
-        public void windowActivated(WindowEvent e) {
-            app = getApplication(e.getWindow());
-            AccessController.doPrivileged(this);
-            app = null;
-        }
-
-        public Object run() {
-            if (app != null) {
-                Thread.currentThread().setContextClassLoader(app.getClassLoader());
-                activeApplication = new WeakReference(app);
-            }
-            else
-                Thread.currentThread().setContextClassLoader(this.getClass().getClassLoader());
-
-            return null;
-        }
-
-        public void windowDeactivated(WindowEvent e) {
-            activeApplication = null;
-        }
-
-        public void windowClosing(WindowEvent e) {
-                System.err.println("Disposing window");
-                e.getWindow().dispose();
-        }
-    };
-
 
     /**
      * Creates a JNLP SecurityManager.
@@ -462,8 +428,6 @@ class JNLPSecurityManager extends Securi
             weakWindows.add(window); // for mapping window -> app
             weakApplications.add(app);
 
-            w.addWindowListener(contextListener); // for dynamic context classloader
-
             app.addWindow(w);
         }