Hi Sergey,

1525         queueEmpty = false;
1526         eventDispatched = false;
1527         synchronized(waitLock) {

Shouldn't lines 1525 & 1526 be moved inside the synchronized block?

--Semyon


On 9/26/2016 8:38 PM, Sergey Bylokhov wrote:
Hello.

Please review the fix for jdk9.

The SunToolkit.waitForIdle() uses string literal for synchronization, which means some code (intentionally or not) can hang Robot.waitForIdle.

Bug: https://bugs.openjdk.java.net/browse/JDK-8166673
Webrev can be found at: http://cr.openjdk.java.net/~serb/8166673/webrev.00


Reply via email to