On Wed, 12 Apr 2023 05:08:46 GMT, Karthik P K <k...@openjdk.org> wrote:

>> When custom skin was loaded, the listeners added in `ContextMenuContent` 
>> class while loading the default skin were not removed. This was causing the 
>> NPE when outdated listeners were invoked.
>> 
>> Updated the code to dispose listeners in the `dispose` method of 
>> `ContextMenuSkin` so that when new skin is loaded, listeners added in the 
>> old skin are removed.
>> 
>> Added system test to validate the fix.
>
> Karthik P K has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Address code review

Marked as reviewed by angorya (Committer).

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

PR Review: https://git.openjdk.org/jfx/pull/1082#pullrequestreview-1381545124

Reply via email to