It looks fine it will be better to dispose the frame in the finally block after the catch. In this case you can call dispose only once.
> > Hi Sergey, > > Thank you for the review comment. Please review the modified code. > http://cr.openjdk.java.net/~mhalder/7119774/webrev.01/ > <http://cr.openjdk.java.net/~mhalder/7119774/webrev.01/> > > Regards, > Manajit > >> On 13-Feb-2017, at 7:20 pm, Sergey Bylokhov <sergey.bylok...@oracle.com >> <mailto:sergey.bylok...@oracle.com>> wrote: >> >> Hi, Manajit. >> Just one comment, the frame should be disposed if the test fails. >> >>> >>> Hi All, >>> >>> Kindly review the fix for JDK9. >>> >>> Bug: >>> https://bugs.openjdk.java.net/browse/JDK-7119774 >>> <https://bugs.openjdk.java.net/browse/JDK-7119774> >>> >>> Webrev: >>> http://cr.openjdk.java.net/~mhalder/7119774/webrev.00/ >>> <http://cr.openjdk.java.net/~mhalder/7119774/webrev.00/> >>> >>> Issue: >>> Test fails sometimes on the last stage because getMousePosition() was >>> getting called before the mouseMove completes. >>> >>> Fix: >>> Fixed the issue by adding a waitForIdle() call after the mouseMove. >>> Along with the fix the following changes were done: >>> The test was cleaned up and moved from closed to open >>> Removed applet code >>> Manual test to automatic test >>> >>> Regards, >>> Manajit >> >
