[icedtea-web] RFC: Rewrite of plugin threading model

Deepak Bhole dbhole at redhat.com
Thu Jun 9 08:55:14 PDT 2011


* Dr Andrew John Hughes <ahughes at redhat.com> [2011-06-08 07:56]:
> On 16:35 Tue 07 Jun     , Deepak Bhole wrote:
> > Hi,
> > 
> > I've been holding back 1.1 for this patch.
> > 
> > This patch adresses PR721:
> > http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=721
> > 
> > The patch in fact fixes far more than just that issue (which have
> > varying symptoms, all ultimately resulting in a browser crash).
> > 
> > The patch rewrites the threading model of the plug-in such that it
> > executes events on the correct thread. Furthermore, it does away with
> > all "chromium" workarounds. With this patch, the plugin works
> > significantly better with chromium. Furthermore, it no longer crashes
> > FF4.
> > 
> > Given the frequency of the newer FF4 crashes, I strongly think this
> > patch should go into 1.1. I have kept things to the absolute minimum and
> > tested it will all known applets and run the live connect tests. I found
> > no regressions vs the current 1.1. The only difference was that there
> > were no crashes, so it is much better with the patch.
> > 
> > Okay for 1.1 and HEAD?
> > 
> 
> Sounds essential for both.  Ok.
> 

Since F15 has 1.0.x and has FF4, I think this patch would make a nice
backport for there too. Okay for 1.0?

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



More information about the distro-pkg-dev mailing list