/hg/release/icedtea-web-1.2: Fix PR1157: Applets can hang browse...

[adomurad at icedtea.classpath.org]

changeset cd4a9f25808e in /hg/release/icedtea-web-1.2
details: http://icedtea.classpath.org/hg/release/icedtea-web-1.2?cmd=changeset;node=cd4a9f25808e
author: Adam Domurad <adomurad at redhat.com>
date: Fri Mar 01 14:22:21 2013 -0500

	Fix PR1157: Applets can hang browser after fatal exception


diffstat:

 ChangeLog                                |   7 +++++++
 NEWS                                     |   3 +++
 netx/net/sourceforge/jnlp/NetxPanel.java |  10 +++++++---
 3 files changed, 17 insertions(+), 3 deletions(-)

diffs (54 lines):

diff -r cd6608773628 -r cd4a9f25808e ChangeLog
--- a/ChangeLog	Mon Jan 07 11:47:37 2013 -0500
+++ b/ChangeLog	Fri Mar 01 14:22:21 2013 -0500
@@ -1,3 +1,10 @@
+2013-03-01 Adam Domurad  <adomurad at redhat.com>
+
+	Fix PR1157: Applets can hang browser after fatal exception
+	* NEWS: Add entry for PR1157
+	* netx/net/sourceforge/jnlp/NetxPanel.java
+	(runLoader): Move dispatchAppletEvent into a 'finally' block.
+
 2013-01-07  Deepak Bhole <dbhole at redhat.com>
 
 	* netx/net/sourceforge/jnlp/resources/Messages.properties: Converted to
diff -r cd6608773628 -r cd4a9f25808e NEWS
--- a/NEWS	Mon Jan 07 11:47:37 2013 -0500
+++ b/NEWS	Fri Mar 01 14:22:21 2013 -0500
@@ -8,6 +8,9 @@
 
 CVE-XXXX-YYYY: http://www.cve.mitre.org/cgi-bin/cvename.cgi?name=XXXX-YYYY
 
+New in release 1.2.4 (2012-XX-XX):
+  - PR1157: Applets can hang browser after fatal exception
+
 New in release 1.2.3 (2012-XX-XX):
 * Common
   - PR1161: X509VariableTrustManager does not work correctly with OpenJDK7
diff -r cd6608773628 -r cd4a9f25808e netx/net/sourceforge/jnlp/NetxPanel.java
--- a/netx/net/sourceforge/jnlp/NetxPanel.java	Mon Jan 07 11:47:37 2013 -0500
+++ b/netx/net/sourceforge/jnlp/NetxPanel.java	Fri Mar 01 14:22:21 2013 -0500
@@ -163,8 +163,6 @@
             // won't make it to the applet, whereas using sun.applet.AppletClassLoader
             // works just fine.
 
-            dispatchAppletEvent(APPLET_LOADING_COMPLETED, null);
-
             if (applet != null) {
                 // Stick it in the frame
                 applet.setStub(this);
@@ -176,7 +174,13 @@
         } catch (Exception e) {
             this.appletAlive = false;
             e.printStackTrace();
-        }
+        } finally {
+            // PR1157: This needs to occur even in the case of an exception
+            // so that the applet's event listeners are signaled.
+            // Once PluginAppletViewer.AppletEventListener is signaled PluginAppletViewer it can properly stop waiting
+            // in PluginAppletViewer.waitForAppletInit
+            dispatchAppletEvent(APPLET_LOADING_COMPLETED, null);
+         }
     }
 
     /**