On Mon, 27 Jul 2026 07:31:38 GMT, Jay Bhaskar <[email protected]> wrote:

> Additional WebKit 625.1 fixes from WebKitGTK 2.52.5
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Change looks good with one inline comment about an unused include and extern 
definition.

I'll complete my testing and then approve.

modules/javafx.web/src/main/native/Tools/DumpRenderTree/java/GCControllerJava.cpp
 line 31:

> 29: #include <jni.h>
> 30: 
> 31: extern "C" JNIEXPORT void JNICALL 
> Java_com_sun_webkit_WebPage_twkDoJSCGarbageCollection(JNIEnv*, jclass);

This is unused, so these 3 lines can be reverted.

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

PR Review: https://git.openjdk.org/jfx/pull/2223#pullrequestreview-4789666761
PR Review Comment: https://git.openjdk.org/jfx/pull/2223#discussion_r3659392682

Reply via email to