On Wed, 22 May 2024 02:51:31 GMT, Alexander Zvegintsev <[email protected]>
wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> spacing
>
> test/jdk/java/awt/List/KeyEventsTest/KeyEventsTest.java line 53:
>
>> 51: import jdk.test.lib.Platform;
>> 52:
>> 53: public class KeyEventsTest {
>
> It is critical to call AWT code on EDT in this, otherwise it may fail with:
>
>> java.lang.NullPointerException: Cannot read field "x" because
>> "this.prevFocusRect" is null
> at
> java.desktop/sun.awt.X11.XListPeer$ListPainter.paintFocus(XListPeer.java:1976)
> at
> java.desktop/sun.awt.X11.XListPeer$ListPainter.paint(XListPeer.java:1799)
> at java.desktop/sun.awt.X11.XListPeer.repaint(XListPeer.java:389)
> at
> java.desktop/sun.awt.X11.XListPeer.scrollVertical(XListPeer.java:1469)
> at java.desktop/sun.awt.X11.XListPeer.makeVisible(XListPeer.java:1311)
> at java.desktop/java.awt.List.makeVisible(List.java:784)
> at KeyEventsTest.test(KeyEventsTest.java:235)
> at KeyEventsTest.doTest(KeyEventsTest.java:292)
> at KeyEventsTest.start(KeyEventsTest.java:107)
> at KeyEventsTest.main(KeyEventsTest.java:56)
> at
> java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
> at java.base/java.lang.reflect.Method.invoke(Method.java:580)
> at
> com.sun.javatest.regtest.agent.MainWrapper$MainTask.run(MainWrapper.java:138)
> at java.base/java.lang.Thread.run(Thread.java:1575)
product bug as well.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19339#discussion_r1744287938
