RFR: 8315655: [macos] Right click and dragging over a component with a popup menu will open the popup [v6]

Damon Nguyen dnguyen at openjdk.org
Fri Jun 7 23:18:12 UTC 2024


On Fri, 7 Jun 2024 21:19:41 GMT, Alisen Chung <achung at openjdk.org> wrote:

>> Issue is a mouse drag will trigger a popup in macos, but not in linux or windows.
>> The solution is to add a check for a mouse pressed event to show popup and prevent mouse entered events from triggering the popup
>
> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update test based on feedback

Tested with and without the fix. Automating it does relieve the need to give a detailed pass/fail condition. Small nit suggestion.

test/jdk/javax/swing/JPopupMenu/MouseDragPopupTest.java line 59:

> 57:             });
> 58:             robot.waitForIdle();
> 59:             robot.delay(1000);

Suggestion:

            robot.setAutoDelay(100);
            robot.setAutoWaitForIdle(true);

            SwingUtilities.invokeAndWait(() -> {
                createAndShowGUI();
            });
            robot.delay(1000);


I was able to remove all the individual `waitForIdle()` calls and replaced it with `setAutoWaitForIdle()`. Still seemed to work fine on macOS.

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

Marked as reviewed by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/19569#pullrequestreview-2105491407
PR Review Comment: https://git.openjdk.org/jdk/pull/19569#discussion_r1631772988


More information about the client-libs-dev mailing list