[icedtea-web] RFC: Make window close events use destroyApplet()

[Deepak Bhole]

Hi,

Attached patch wires window close events to use destroyApplet() rather
than appletClose(). 

destroyApplet() calls appletClose(), but is thread-safe and can be called
multiple times safely unlike appletClose() which throws an NPE if
called again (depending on timing).

I think this patch should go into 1.0 as well. OK for both?

ChangeLog:
2011-04-07  Deepak Bhole <dbhole at redhat.com>

    * plugin/icedteanp/java/sun/applet/PluginAppletViewer.java
    (constructor): Make window close event call destroy applet which can be
    safely called multiple times, unlike appletClose.

Cheers,
Deepak
-------------- next part --------------
diff -r 9c89214b7ef6 plugin/icedteanp/java/sun/applet/PluginAppletViewer.java
--- a/plugin/icedteanp/java/sun/applet/PluginAppletViewer.java	Tue Apr 05 12:39:14 2011 -0400
+++ b/plugin/icedteanp/java/sun/applet/PluginAppletViewer.java	Thu Apr 07 17:36:36 2011 -0400
@@ -407,7 +407,7 @@
         windowEventListener = new WindowAdapter() {
 
             public void windowClosing(WindowEvent evt) {
-                appletClose();
+                destroyApplet(identifier);
             }
 
             public void windowIconified(WindowEvent evt) {