On Mon, 18 Aug 2025 16:03:23 GMT, Anass Baya <ab...@openjdk.org> wrote:

> This test was recently automated and is failing intermittently in the CI due 
> to timing issues.
> This enhancement aims to stabilize the test and also adds the missing null 
> pointer check.

test/jdk/javax/swing/DataTransfer/bug4655513.java line 73:

> 71:                 robot.delay(20);
> 72:             }
> 73:             robot.mouseRelease(InputEvent.BUTTON1_DOWN_MASK);

mousePress parameter should be `InputEvent.BUTTON1_DOWN_MASK` instead which is 
usually the norm? mouseRelease is called correctly..
Also, I dont think we usually use delay between drag...guess once drag is 
complete, we can call waitForIdle and delay..

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26824#discussion_r2287476781

Reply via email to