On Sat, 26 Apr 2025 02:56:55 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> Manukumar V S has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review comments fixed : Code reformatting, changed the indentation to the 
>> standard four-space indentation
>
> test/jdk/java/awt/ScrollPane/bug8077409Test.java line 69:
> 
>> 67:         try {
>> 68:             obj.setLocationRelativeTo(null);
>> 69:             Toolkit.getDefaultToolkit().addAWTEventListener(e -> {
> 
> This test is not manual and does not use a robot to click. How can this 
> listener be called?

Yea, that code piece was unwanted, I have removed it and kept it in a clean way.

I have also tested this with the latest changes on Windows-x64 with JDK 
8u60_b04 and JDK 8u60_b20 as the original issue related to this 
test(JDK-8077409) was raised for Windows-x64 and was fixed in JDK 8u60b20.
Results:

With JDK 8u60b04:
Exception in thread "main" java.lang.RuntimeException: Wrong position of 
component in ScrollPane
        at bug8077409Test.main(bug8077409Test.java:142)

With JDK 8u60b20:
Passed.....

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24292#discussion_r2063300548

Reply via email to