On Mon, 16 Sep 2024 18:45:45 GMT, Hima Bindu Meda <hm...@openjdk.org> wrote:
> Updated additional webkit-619.1 fixes from webkitgtk-2.44.4. Build is > successful on all platforms.No issues seen All my testing looks good. The code changes look good with one question that I left. modules/javafx.web/src/main/native/Source/WebKitLegacy/java/WebCoreSupport/HistoryItemClientJava.cpp line 1: > 1: #include "HistoryItemClientJava.h" Where did this file come from? I don't see any changes related to HistoryItemClient in the 2.44.4 release of WebKitGTK? Presuming we need this file, it will need a proper copyright header. modules/javafx.web/src/main/native/Source/WebKitLegacy/java/WebCoreSupport/HistoryItemClientJava.h line 1: > 1: #pragma once Presuming we need this file, it will need a proper copyright header. ------------- PR Review: https://git.openjdk.org/jfx/pull/1570#pullrequestreview-2312524694 PR Review Comment: https://git.openjdk.org/jfx/pull/1570#discussion_r1764957413 PR Review Comment: https://git.openjdk.org/jfx/pull/1570#discussion_r1764957887