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

Reply via email to