<AWT Dev> [9] Review Request: JDK-8037840 [macosx] Rewrite CWarning window to eliminate the ExecutorService

Petr Pchelko petr.pchelko at oracle.com
Wed Mar 19 14:15:31 UTC 2014

Hello, AWT Team.

Please review the fix for the issue: https://bugs.openjdk.java.net/browse/JDK-8037840
The fix is available at: http://cr.openjdk.java.net/~pchelko/9/8037840/webrev/

This is another fix for the SecurityWarning window. The problem is that we create a new Java thread to animate the warning icon. And we have a thread per icon. 
That's an overkill and a waste of resources. The idea of this fix is to use Appkit to schedule delayed tasks. 

You might wonder, why I'm rewriting the code I've just fixed in JDK-8037776. The idea is that JDK-8037776 is a completely safe fix to backport it to 8u20. This fix is a bit more risky, so it will stay in 9 without a backport to 8u20.

More information about the awt-dev mailing list