From fe3cbabc31d8942ff3555f513f197536fbc52c16 Mon Sep 17 00:00:00 2001 From: Nirupam Bhowmick <48842933+heydryft@users.noreply.github.com> Date: Thu, 25 Dec 2025 22:00:27 +0000 Subject: [PATCH] fix: update gfxTextRun.cpp constructor hunks for Firefox 146 - Removed duplicate mUserContextId(0) line - Removed obsolete switch(aVariantEmoji) context (removed in FF146) - Removed mCurrGeneration = 0 (not in FF146 constructor) - Fixed hunk header line count from 10 to 6 - Simplified mFontVisibilityProvider null check - gfxTextRun.cpp now patches cleanly - 8 rejects remaining (down from 11) --- patches/roverfox/font-fingerprinting.context.patch | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/patches/roverfox/font-fingerprinting.context.patch b/patches/roverfox/font-fingerprinting.context.patch index cd7393ab..b9efa457 100644 --- a/patches/roverfox/font-fingerprinting.context.patch +++ b/patches/roverfox/font-fingerprinting.context.patch @@ -868,25 +868,19 @@ index a8fac45e67..d59815f2d5 100644 EmphasisMarkDrawParams params(aContext, aPaletteCache); params.mark = aMark; -@@ -1873,6 +1877,8 @@ gfxFontGroup::gfxFontGroup(FontVisibilityProvider* aFontVisibilityProvider, +@@ -1872,6 +1876,22 @@ gfxFontGroup::gfxFontGroup(FontVisibilityProvider* aFontVisibilityProvider, mPageLang(gfxPlatformFontList::GetFontPrefLangFor(aLanguage)), mExplicitLanguage(aExplicitLanguage), - mFontVariantEmoji(aVariantEmoji) { + mFontVariantEmoji(aVariantEmoji), + mUserContextId(0) { -+ mUserContextId(0) { - switch (aVariantEmoji) { - case StyleFontVariantEmoji::Normal: - case StyleFontVariantEmoji::Unicode: -@@ -1875,7 +1880,19 @@ gfxFontGroup::gfxFontGroup(FontVisibilityProvider* aFontVisibilityProvider, // We don't use SetUserFontSet() here, as we want to unconditionally call // EnsureFontList() rather than only do UpdateUserFonts() if it changed. - mCurrGeneration = 0; + + // Initialize cached user context ID from the document principal. + mUserContextId = 0; + if (mFontVisibilityProvider) { - if (auto* doc = mFontVisibilityProvider ? mFontVisibilityProvider->GetDocument() : nullptr) { ++ if (auto* doc = mFontVisibilityProvider->GetDocument()) { + if (auto* principal = doc->NodePrincipal()) { + auto* bp = mozilla::BasePrincipal::Cast(principal); + if (bp) {