On Wed, 9 Nov 2022 10:37:24 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
> Test intermittently fails in VM citing "Tree is not editing". Seems to be > problem with mouse clicks not getting registered properly.. > Similar test test/jdk/javax/swing/JTable/6263446/bug6263446.java is not > affected, so made the test similar to it by using same safeguard using > Robot.waitForIdle() and modifying clickpoint to tree cell midpoint. > > Several iterations of the test pass in the OCI VM and all other physical > platforms (link in JBS) test/jdk/javax/swing/JTree/6263446/bug6263446.java line 131: > 129: robot.mousePress(InputEvent.BUTTON1_DOWN_MASK); > 130: robot.mouseRelease(InputEvent.BUTTON1_DOWN_MASK); > 131: robot.waitForIdle(); Why do we remove it here and then add it after each call of this "click" method? why not move to the end of the method? test/jdk/javax/swing/JTree/6263446/bug6263446.java line 186: > 184: frame.setAlwaysOnTop(true); > 185: frame.setLocationRelativeTo(null); > 186: frame.pack(); pack() after the setLocationRelativeTo() may move the frame out of the center of the screen, is it intentional? ------------- PR: https://git.openjdk.org/jdk/pull/11057
