On Thu, 1 Feb 2024 20:52:14 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> SunToolkit.java is trying to post an event on the TrayIcon appContext, but >> the TrayIcon was already removed by the test, causing an error. The fix is >> to make SunToolkit skip posting the event if appContext is null. The test is >> also updated to remove applet usage and use PassFailJFrame instead. > > Alisen Chung has updated the pull request incrementally with two additional > commits since the last revision: > > - remove change in SunToolkit > - correct null change src/java.desktop/macosx/classes/sun/lwawt/macosx/CTrayIcon.java line 176: > 174: > 175: LWCToolkit.targetDisposedPeer(target, this); > 176: target = null; to make sure it is not changed in some other place you can merk the target field as final. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17329#discussion_r1479107367