RFR: 8088420: JavaFX WebView memory leak via EventListener [v4]

[Jay Bhaskar]

On Tue, 24 May 2022 14:54:19 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adding JGObject in plave of raw jni object
>
> modules/javafx.web/src/main/native/Source/WebCore/bindings/java/EventListenerManager.cpp line 99:
> 
>> 97: }
>> 98: 
>> 99: void EventListenerManager::resetDOMWindow(DOMWindow* window)
> 
> Have you validated this logic? If I understand correctly, the `isReferringToOtherListener` var will be true if any listener in the list of listeners for this DOM window has a ref count > 1. Is my understanding correct? This doesn't seem quite right to me, but I may be missing something.

yes , this is the case where we would remove pairs from map.

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

PR: https://git.openjdk.java.net/jfx/pull/799