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.