Remove CPDF_RenderStatus::GetFillArgbInternal(). The bool that changes its behaviour is only used very early on, so combine with its callers. Change-Id: I63160b11f8fe0949fe76ad889f1098a3e42d3ac3 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/82192 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
diff --git a/core/fpdfapi/render/cpdf_renderstatus.cpp b/core/fpdfapi/render/cpdf_renderstatus.cpp index daca30b..dd7b67c 100644 --- a/core/fpdfapi/render/cpdf_renderstatus.cpp +++ b/core/fpdfapi/render/cpdf_renderstatus.cpp
@@ -442,12 +442,15 @@ return pDocCache ? pDocCache->GetTransferFunc(pObj) : nullptr; } -FX_ARGB CPDF_RenderStatus::GetFillArgbInternal(CPDF_PageObject* pObj, - bool bType3) const { - const CPDF_ColorState* pColorState = &pObj->m_ColorState; - if (!bType3 && Type3CharMissingFillColor(m_pType3Char.Get(), pColorState)) +FX_ARGB CPDF_RenderStatus::GetFillArgb(CPDF_PageObject* pObj) const { + if (Type3CharMissingFillColor(m_pType3Char.Get(), &pObj->m_ColorState)) return m_T3FillColor; + return GetFillArgbForType3(pObj); +} + +FX_ARGB CPDF_RenderStatus::GetFillArgbForType3(CPDF_PageObject* pObj) const { + const CPDF_ColorState* pColorState = &pObj->m_ColorState; if (MissingFillColor(pColorState)) pColorState = &m_InitialStates.m_ColorState;
diff --git a/core/fpdfapi/render/cpdf_renderstatus.h b/core/fpdfapi/render/cpdf_renderstatus.h index d455ca6..f8b31d4 100644 --- a/core/fpdfapi/render/cpdf_renderstatus.h +++ b/core/fpdfapi/render/cpdf_renderstatus.h
@@ -101,12 +101,8 @@ RetainPtr<CPDF_TransferFunc> GetTransferFunc( const CPDF_Object* pObject) const; - FX_ARGB GetFillArgb(CPDF_PageObject* pObj) const { - return GetFillArgbInternal(pObj, false); - } - FX_ARGB GetFillArgbForType3(CPDF_PageObject* pObj) const { - return GetFillArgbInternal(pObj, true); - } + FX_ARGB GetFillArgb(CPDF_PageObject* pObj) const; + FX_ARGB GetFillArgbForType3(CPDF_PageObject* pObj) const; void DrawTilingPattern(CPDF_TilingPattern* pPattern, CPDF_PageObject* pPageObj, @@ -129,7 +125,6 @@ bool stroke); private: - FX_ARGB GetFillArgbInternal(CPDF_PageObject* pObj, bool bType3) const; bool ProcessTransparency(CPDF_PageObject* PageObj, const CFX_Matrix& mtObj2Device); void ProcessObjectNoClip(CPDF_PageObject* pObj,