[rfc][icedtea-web] Kill processes in Integration Tests more cleanly

Jie Kang jkang at redhat.com
Thu Apr 9 19:58:40 UTC 2015


This patch does four things:

1. Removes the usage of the softkiller : it no longer closes tabs properly and requires very old 3rd-party dependencies
2. Changes the signal sent from SIGINT to SIGUSR1 : mainly for firefox which doesn't handle any of the signals very well : https://bugzilla.mozilla.org/show_bug.cgi?id=336193
    Afaik the other browsers handle this signal by closing. I haven't been able to find any documentation on whether or not they close gracefully though.
    SIGTERM would be okay too
3. Removes the usage of p.destroy() : it may or may not forcibly kill the process depending on the implementation and with the signal sent above, shouldn't be necessary
4. Lowers the timeout time from 20 seconds to 10 seconds 

In general, it should make the suite go faster and remove issues that come from the browser and applet being destroyed without letting them clean up.

I have run the reproducer suite on my machine (F21, openjdk 1.8) and there haven't been any new failing tests while these changes were applied. Technically, my timeout was at 5 seconds, not 10, but I think 10 is more reasonable given that some of the tests involve downloading from a remote source. To be honest though, I'd prefer it if each individual test was able to run in under 5 seconds...

If others could take this patch, apply it and see how the reproducers fair for them, it'd be greatly appreciated.



Jie Kang

OpenJDK Team - Software Engineering Intern
-------------- next part --------------
A non-text attachment was scrubbed...
Name: itw-integration-kill-cleanly-1.patch
Type: text/x-patch
Size: 3530 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20150409/5577f68f/itw-integration-kill-cleanly-1.patch>

More information about the distro-pkg-dev mailing list