On Wed, 10 Jan 2024 21:08:35 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: > > - spacing > - updated test title, copyright year, removed redundant check Extra blank lines can be removed else looks good to me. Since `html` file has been removed, I guess the java file can be taken out of `ShowAfterDisposeTest` folder. ------------- Marked as reviewed by abhiscxk (Committer). PR Review: https://git.openjdk.org/jdk/pull/17329#pullrequestreview-1817331918 PR Comment: https://git.openjdk.org/jdk/pull/17329#issuecomment-1888406196
