On Fri, 16 Jan 2026 06:58:40 GMT, Hima Bindu Meda <[email protected]> wrote:
> This PR updates Webkit to v623.1. Verified build and sanity on Mac, Linux and
> Windows platforms. No issue seen.
@HimaBinduMeda There is a minor merge issue that will need to be fixed up
following the recent integration of the changes from PR #2047 .
To address this, do the following:
1. Merge in the latest jfx master.
2. Apply the following patch:
---
a/modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/ComplexTextController.cpp
+++
b/modules/javafx.web/src/main/native/Source/WebCore/platform/graphics/ComplexTextController.cpp
@@ -499,7 +499,7 @@ void
ComplexTextController::ComplexTextRun::setIsNonMonotonic()
#else
for (unsigned i = 0; i < m_glyphCount; ++i) {
// Skip zero-width glyphs if needed
- if (m_font.isZeroWidthSpaceGlyph(m_glyphs[i]))
+ if (m_font->isZeroWidthSpaceGlyph(m_glyphs[i]))
continue;
// Handle out of bound case
unsigned charIndex = indexAt(i);
I've already tested it both locally, and in our CI, so I'll just run a sanity
test once you update the PR.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/2042#issuecomment-3774287766
