RFR: 8315986: javax/swing/JMenuItem/4654927/bug4654927.java: component must be showing on the screen to determine its location [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Wed Oct 4 02:59:30 UTC 2023


On Tue, 3 Oct 2023 21:24:59 GMT, Phil Race <prr at 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


More information about the client-libs-dev mailing list