On Thu, 1 Feb 2024 20:52:14 GMT, Alisen Chung <[email protected]> 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

If you change content of CTrayIcon.java it is possible a good idea to bump up 
the copyright year to 2024. Aside of that looks like all review comments were 
addressed.

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

Marked as reviewed by kizune (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/17329#pullrequestreview-1864029520

Reply via email to