RFR: 8353549: Open source events tests batch2 [v2]

Alexander Zvegintsev azvegint at openjdk.org
Tue Apr 8 23:51:58 UTC 2025


On Tue, 8 Apr 2025 20:39:14 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comments
>
> test/jdk/java/awt/event/MouseEvent/MouseRButTest.java line 61:
> 
>> 59:             robot.delay(500);
>> 60: 
>> 61:             Point point = Util.invokeOnEDT(() -> button.getLocationOnScreen());
> 
> Received a suggestion on one of my other PRs that awt/regtesthelper/Util can be replaced with swing/regtesthelper/Util.getCenterPoint(button) (makes it easier and it is already called on EDT)

Yes, sometimes it may be helpful

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24516#discussion_r2034177295


More information about the client-libs-dev mailing list