Looks fine although fix the formatting here where there is a missing space before catch :

+        }catch (InterruptedException ex) {


-phil.

On 1/18/18, 10:44 AM, Krishna Addepalli wrote:

Hi All,

Please review a fix for following bug:

Bug: JDK-8137101 : https://bugs.openjdk.java.net/browse/JDK-8137101

Webrev: http://cr.openjdk.java.net/~kaddepalli/8137101/webrev00/ <http://cr.openjdk.java.net/%7Ekaddepalli/8137101/webrev00/>

The problem is that EDT doesnot get enough time to show the window on screen before the next function tries to pick colors from the screen. The solution is to use CountDownLatch to make the main thread wait deterministically before trying to pick colors.

Thanks,

Krishna

Reply via email to