From 8f63763317ae1ef65d1ac2719624adeb728f2207 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 7 May 2018 23:36:26 +0000 Subject: Make more CPDF_PageObjectHolder members protected. Change-Id: Id31b9f6ac33971c47cdb378ff7f9ca7d33f899b9 Reviewed-on: https://pdfium-review.googlesource.com/32112 Commit-Queue: Lei Zhang Reviewed-by: dsinclair --- core/fpdfapi/edit/cpdf_pagecontentgenerator.cpp | 2 +- core/fpdfapi/page/cpdf_contentparser.cpp | 8 ++++---- core/fpdfapi/page/cpdf_pageobjectholder.h | 4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) (limited to 'core/fpdfapi') diff --git a/core/fpdfapi/edit/cpdf_pagecontentgenerator.cpp b/core/fpdfapi/edit/cpdf_pagecontentgenerator.cpp index 92b92ecf01..e9b134fa6b 100644 --- a/core/fpdfapi/edit/cpdf_pagecontentgenerator.cpp +++ b/core/fpdfapi/edit/cpdf_pagecontentgenerator.cpp @@ -52,7 +52,7 @@ bool GetColor(const CPDF_Color* pColor, float* rgb) { CPDF_PageContentGenerator::CPDF_PageContentGenerator( CPDF_PageObjectHolder* pObjHolder) - : m_pObjHolder(pObjHolder), m_pDocument(pObjHolder->m_pDocument.Get()) { + : m_pObjHolder(pObjHolder), m_pDocument(pObjHolder->GetDocument()) { for (const auto& pObj : *pObjHolder->GetPageObjectList()) { if (pObj) m_pageObjects.emplace_back(pObj.get()); diff --git a/core/fpdfapi/page/cpdf_contentparser.cpp b/core/fpdfapi/page/cpdf_contentparser.cpp index 0b11da5554..dcdf138779 100644 --- a/core/fpdfapi/page/cpdf_contentparser.cpp +++ b/core/fpdfapi/page/cpdf_contentparser.cpp @@ -24,7 +24,7 @@ CPDF_ContentParser::CPDF_ContentParser(CPDF_Page* pPage) : m_InternalStage(STAGE_GETCONTENT), m_pObjectHolder(pPage) { - if (!pPage || !pPage->m_pDocument || !pPage->m_pFormDict) { + if (!pPage || !pPage->GetDocument() || !pPage->m_pFormDict) { m_bIsDone = true; return; } @@ -84,7 +84,7 @@ CPDF_ContentParser::CPDF_ContentParser(CPDF_Form* pForm, CPDF_Dictionary* pResources = pForm->m_pFormDict->GetDictFor("Resources"); m_pParser = pdfium::MakeUnique( - pForm->m_pDocument.Get(), pForm->m_pPageResources.Get(), + pForm->GetDocument(), pForm->m_pPageResources.Get(), pForm->m_pResources.Get(), pParentMatrix, pForm, pResources, form_bbox, pGraphicStates, parsedSet); m_pParser->GetCurStates()->m_CTM = form_matrix; @@ -157,10 +157,10 @@ bool CPDF_ContentParser::Continue(PauseIndicatorIface* pPause) { if (!m_pParser) { m_parsedSet = pdfium::MakeUnique>(); m_pParser = pdfium::MakeUnique( - m_pObjectHolder->m_pDocument.Get(), + m_pObjectHolder->GetDocument(), m_pObjectHolder->m_pPageResources.Get(), nullptr, nullptr, m_pObjectHolder.Get(), m_pObjectHolder->m_pResources.Get(), - m_pObjectHolder->m_BBox, nullptr, m_parsedSet.get()); + m_pObjectHolder->GetBBox(), nullptr, m_parsedSet.get()); m_pParser->GetCurStates()->m_ColorState.SetDefault(); } if (m_CurrentOffset >= m_Size) { diff --git a/core/fpdfapi/page/cpdf_pageobjectholder.h b/core/fpdfapi/page/cpdf_pageobjectholder.h index 98cd97adb2..54c4f2b063 100644 --- a/core/fpdfapi/page/cpdf_pageobjectholder.h +++ b/core/fpdfapi/page/cpdf_pageobjectholder.h @@ -89,18 +89,18 @@ class CPDF_PageObjectHolder { const UnownedPtr m_pFormDict; UnownedPtr m_pFormStream; - UnownedPtr m_pDocument; UnownedPtr m_pPageResources; UnownedPtr m_pResources; std::map m_GraphicsMap; std::map m_FontsMap; - CFX_FloatRect m_BBox; protected: enum ParseState { CONTENT_NOT_PARSED, CONTENT_PARSING, CONTENT_PARSED }; void LoadTransInfo(); + UnownedPtr m_pDocument; + CFX_FloatRect m_BBox; int m_iTransparency = 0; bool m_bBackgroundAlphaNeeded = false; std::vector m_MaskBoundingBoxes; -- cgit v1.2.3