On Tue, 3 Oct 2023 21:24:59 GMT, Phil Race <p...@openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add setAutoWaitForIdle
>
> test/jdk/javax/swing/JMenuItem/4654927/bug4654927.java line 83:
> 
>> 81:             robot.mouseRelease(InputEvent.BUTTON1_DOWN_MASK);
>> 82:             robot.waitForIdle();
>> 83:             robot.delay(250);
> 
> These don't help on the system I have tested on.
> There's something else going on.
> The test is basically about verifying that clicking on a disabled menu item 
> doesn't dismiss the menu.
> I don't think it does. If I comment out the press/release the same exception 
> occurs.
> And visually you can see the menu disappear and then reappear.
> What it looks like is that the call at line 86/91 to mouseMove which returns 
> the pointer to the "menu" location causes the list of menu items to disappear 
> and then quickly reappear. Sampling if the menu is on screen at that time 
> will cause the error.
> It only happens once .. very odd ..

Can you please tell which system it is not working?
 I guess harshita and vitaly tested on macos14 and it worked for them and I 
tested on other mac13,windows10 locally where it also worked, as well as in 
mach5 systems, whose link I have put in JBS.

I also have seen the menu disappear while clicking but it does not cause the 
test to fail..I saw the video recording vitaly uploaded in JBS and it is same 
as in macos13 where it pass always (we also dont see any failure for this test 
in CI runs so far)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15677#discussion_r1345112716

Reply via email to