RFR: 8368572: Update WebKit to 623.1
Kevin Rushforth
kcr at openjdk.org
Tue Jan 20 18:18:13 UTC 2026
On Fri, 16 Jan 2026 06:58:40 GMT, Hima Bindu Meda <hmeda at openjdk.org> 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
More information about the openjfx-dev
mailing list