[icedtea-web] RFC: Request to back out 0256de6a4bf6

[Deepak Bhole]

* Dr Andrew John Hughes <ahughes at redhat.com> [2011-05-16 20:33]:
> On 13:55 Mon 16 May     , Deepak Bhole wrote:
> > Hi,
> > 
> > http://icedtea.classpath.org/hg/release/icedtea-web-1.1/rev/0256de6a4bf6
> > makes it so that the VM shuts down after the last applet is closed.
> > 
> > The patch was committed with the known possibility that there could be a
> > race condition whereby one applet was initializing while the last one
> > closed. The probability of that was deemed to be low enough to not
> > matter.
> > 
> > However I think that assessment was incorrect on my part. I have come
> > across pages where one applet tries to launch a page that loads another.
> > In such case of programmatic switching, it is easier to end up with a
> > case where the VM shuts down when it shouldn't.
> > 
> > I would therefore like to back this patch out of 1.1.
> > 
> > The proper fix is to do shutdown from C++ side, which can then also
> > bring the VM back up if another applet is loaded. I have a patch do
> > this, but it will need some more testing and it shouldn't go in 1.1. I
> > will apply it to head after I am done testing it.
> > 
> > Okay to back out and replace with just commenting out tg.stop()?
> > 
> > Doing so may leak resources due to bad applets, but that is not really
> > the fault of the plug-in anyway.
> > 
> > Cheers,
> > Deepak
> 
> Approved.  Do we have a new release date for 1.1?

Yes, I sent an email earlier today:
http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2011-May/014245.html

Cheers,
Deepak

> -- 
> Andrew :)
> 
> Free Java Software Engineer
> Red Hat, Inc. (http://www.redhat.com)
> 
> Support Free Java!
> Contribute to GNU Classpath and IcedTea
> http://www.gnu.org/software/classpath
> http://icedtea.classpath.org
> PGP Key: F5862A37 (https://keys.indymedia.org/)
> Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37