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

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Wed Mar 19 14:31:37 UTC 2014

Hi, Petr.
The fix looks good.

On 3/19/14 6:15 PM, Petr Pchelko wrote:
> 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.

Best regards, Sergey.

More information about the awt-dev mailing list