<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi, Manajit.<div class="">Just one comment, the frame should be disposed if the test fails.</div><div class=""><br class=""><div><blockquote type="cite" class=""><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="line-height: 16px;" class=""><font size="2" class="">Hi All,</font></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Kindly review the fix for JDK9.</font></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Bug: </font></div><div style="line-height: 16px;" class=""><font size="2" class=""><a href="https://bugs.openjdk.java.net/browse/JDK-7119774" class="">https://bugs.openjdk.java.net/browse/JDK-7119774</a></font></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Webrev: </font></div><div style="line-height: 16px;" class=""><font size="2" class=""><a href="http://cr.openjdk.java.net/~mhalder/7119774/webrev.00/" class="">http://cr.openjdk.java.net/~mhalder/7119774/webrev.00/</a></font></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Issue: </font></div><div style="line-height: 16px;" class=""><font size="2" class="">Test <span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class="">fails sometimes on the last stage because getMousePosition() was getting called before the mouseMove completes.</span><span style="color: rgb(51, 51, 51); background-color: rgb(255, 255, 255);" class=""> </span></font></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Fix: </font></div><div style="line-height: 16px;" class=""><div style="margin: 0px; line-height: normal;" class=""><font size="2" class="">Fixed the issue by adding a waitForIdle() call after the mouseMove.</font></div><div style="margin: 0px; line-height: normal;" class=""><font size="2" class="">Along with the fix the following changes were done:</font></div><div style="margin: 0px; line-height: normal;" class=""><font size="2" class=""><span class="Apple-tab-span" style="white-space: pre;">  </span>The test was cleaned up and moved from closed to open</font></div><div style="margin: 0px; line-height: normal;" class=""><font size="2" class=""><span class="Apple-tab-span" style="white-space: pre;">        </span>Removed applet code</font></div><div style="margin: 0px; line-height: normal;" class=""><font size="2" class=""><span class="Apple-tab-span" style="white-space: pre;">  </span>Manual test to automatic test</font></div></div><div style="line-height: 16px;" class=""><font size="2" class=""><br class=""></font></div><div style="line-height: 16px;" class=""><font size="2" class="">Regards,</font></div><div style="line-height: 16px;" class=""><font size="2" class="">Manajit</font></div></div></div></blockquote></div><br class=""></div></body></html>