Looks good to me.
Regards, Rajeev Chamyal From: Avik Niyogi Sent: 31 May 2016 13:28 To: Rajeev Chamyal; Alexander Scherbatiy; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> Review request for 8132771: [TEST_BUG][macosx] Test javax/swing/JTree/DnD/LastNodeLowerHalfDrop.java fails for MacOSX Hi All, Please review the code changes with inputs provided. http://cr.openjdk.java.net/~aniyogi/8132771/webrev.01/ With Regards, Avik Niyogi On 31-May-2016, at 1:24 pm, Rajeev Chamyal <HYPERLINK "mailto:rajeev.cham...@oracle.com"rajeev.cham...@oracle.com> wrote: Hello Avik, The frame should be disposed in case of exception also. Regards, Rajeev Chamyal From: Avik Niyogi Sent: 31 May 2016 11:14 To: Rajeev Chamyal; Alexander Scherbatiy; HYPERLINK "mailto:swing-dev@openjdk.java.net"swing-dev@openjdk.java.net Subject: <Swing Dev> Review request for 8132771: [TEST_BUG][macosx] Test javax/swing/JTree/DnD/LastNodeLowerHalfDrop.java fails for MacOSX Hi All, Kindly review the fix for JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8132771 Webrev: http://cr.openjdk.java.net/~aniyogi/8132771/webrev.00/ Issue: LastNodeLowerHalfDrop Test case throws an exception when the behaviour is as expected. Cause: The robot does not have enough delay to run appropriately. Fix: The test case was fixed to have enough delay for the test case. With Regards, Avik Niyogi