summaryrefslogtreecommitdiff
path: root/core/fpdfapi
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2018-05-15 17:12:15 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-05-15 17:12:15 +0000
commit68c15ce8891f7b65234940d29849f8f0af91e3c6 (patch)
tree973f20d2b69651cccfa426b2bbefd7f92ac1251e /core/fpdfapi
parent473b4ae489e8c6d63b087693589eda03399f60f6 (diff)
downloadpdfium-68c15ce8891f7b65234940d29849f8f0af91e3c6.tar.xz
Break CPDF_CotnentParser::Continue into pieces
This CL separates the parts of CPDF_ContentParser::Continue into three distinct methods. It clarifes when and what is returned from Continue. Change-Id: Id7a9bf1aa16f366f567cce74006c27c452cb5dcc Reviewed-on: https://pdfium-review.googlesource.com/32531 Reviewed-by: Henrique Nakashima <hnakashima@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core/fpdfapi')
-rw-r--r--core/fpdfapi/page/cpdf_contentparser.cpp190
-rw-r--r--core/fpdfapi/page/cpdf_contentparser.h4
2 files changed, 109 insertions, 85 deletions
diff --git a/core/fpdfapi/page/cpdf_contentparser.cpp b/core/fpdfapi/page/cpdf_contentparser.cpp
index 26fd0189f8..93c64a14ec 100644
--- a/core/fpdfapi/page/cpdf_contentparser.cpp
+++ b/core/fpdfapi/page/cpdf_contentparser.cpp
@@ -109,99 +109,119 @@ CPDF_ContentParser::CPDF_ContentParser(CPDF_Form* pForm,
CPDF_ContentParser::~CPDF_ContentParser() {}
+// Returning |true| means that there is more content to be processed and
+// Continue() should be called again. Returning |false| means that we've
+// completed the parse and Continue() is complete.
bool CPDF_ContentParser::Continue(PauseIndicatorIface* pPause) {
if (m_bIsDone)
return false;
- while (!m_bIsDone) {
- if (m_InternalStage == STAGE_GETCONTENT) {
- if (m_CurrentOffset == m_nStreams) {
- if (!m_StreamArray.empty()) {
- FX_SAFE_UINT32 safeSize = 0;
- for (const auto& stream : m_StreamArray) {
- safeSize += stream->GetSize();
- safeSize += 1;
- }
- if (!safeSize.IsValid()) {
- m_bIsDone = true;
- return false;
- }
- m_Size = safeSize.ValueOrDie();
- m_pData.Reset(std::unique_ptr<uint8_t, FxFreeDeleter>(
- FX_Alloc(uint8_t, m_Size)));
- uint32_t pos = 0;
- for (const auto& stream : m_StreamArray) {
- memcpy(m_pData.Get() + pos, stream->GetData(), stream->GetSize());
- pos += stream->GetSize();
- m_pData.Get()[pos++] = ' ';
- }
- m_StreamArray.clear();
- } else {
- m_pData.Reset(m_pSingleStream->GetData());
- m_Size = m_pSingleStream->GetSize();
- }
- m_InternalStage = STAGE_PARSE;
- m_CurrentOffset = 0;
- } else {
- CPDF_Array* pContent =
- m_pObjectHolder->GetFormDict()->GetArrayFor("Contents");
- CPDF_Stream* pStreamObj = ToStream(
- pContent ? pContent->GetDirectObjectAt(m_CurrentOffset) : nullptr);
- m_StreamArray[m_CurrentOffset] =
- pdfium::MakeRetain<CPDF_StreamAcc>(pStreamObj);
- m_StreamArray[m_CurrentOffset]->LoadAllDataFiltered();
- m_CurrentOffset++;
- }
+ while (true) {
+ switch (m_InternalStage) {
+ case STAGE_GETCONTENT:
+ StageGetContent();
+ break;
+ case STAGE_PARSE:
+ StageParse();
+ break;
+ case STAGE_CHECKCLIP:
+ StageCheckClip();
+ break;
}
- if (m_InternalStage == STAGE_PARSE) {
- if (!m_pParser) {
- m_parsedSet = pdfium::MakeUnique<std::set<const uint8_t*>>();
- m_pParser = pdfium::MakeUnique<CPDF_StreamContentParser>(
- m_pObjectHolder->GetDocument(),
- m_pObjectHolder->m_pPageResources.Get(), nullptr, nullptr,
- m_pObjectHolder.Get(), m_pObjectHolder->m_pResources.Get(),
- m_pObjectHolder->GetBBox(), nullptr, m_parsedSet.get());
- m_pParser->GetCurStates()->m_ColorState.SetDefault();
- }
- if (m_CurrentOffset >= m_Size) {
- m_InternalStage = STAGE_CHECKCLIP;
- } else {
- m_CurrentOffset +=
- m_pParser->Parse(m_pData.Get() + m_CurrentOffset,
- m_Size - m_CurrentOffset, PARSE_STEP_LIMIT);
+
+ if (m_bIsDone)
+ return false;
+ if (pPause && pPause->NeedToPauseNow())
+ return true;
+ }
+}
+
+void CPDF_ContentParser::StageGetContent() {
+ if (m_CurrentOffset == m_nStreams) {
+ if (!m_StreamArray.empty()) {
+ FX_SAFE_UINT32 safeSize = 0;
+ for (const auto& stream : m_StreamArray) {
+ safeSize += stream->GetSize();
+ safeSize += 1;
}
- }
- if (m_InternalStage == STAGE_CHECKCLIP) {
- if (m_pType3Char) {
- m_pType3Char->InitializeFromStreamData(m_pParser->IsColored(),
- m_pParser->GetType3Data());
+ if (!safeSize.IsValid()) {
+ m_bIsDone = true;
+ return;
}
-
- for (auto& pObj : *m_pObjectHolder->GetPageObjectList()) {
- if (!pObj->m_ClipPath.HasRef())
- continue;
- if (pObj->m_ClipPath.GetPathCount() != 1)
- continue;
- if (pObj->m_ClipPath.GetTextCount() > 0)
- continue;
-
- CPDF_Path ClipPath = pObj->m_ClipPath.GetPath(0);
- if (!ClipPath.IsRect() || pObj->IsShading())
- continue;
-
- CFX_PointF point0 = ClipPath.GetPoint(0);
- CFX_PointF point2 = ClipPath.GetPoint(2);
- CFX_FloatRect old_rect(point0.x, point0.y, point2.x, point2.y);
- CFX_FloatRect obj_rect(pObj->m_Left, pObj->m_Bottom, pObj->m_Right,
- pObj->m_Top);
- if (old_rect.Contains(obj_rect))
- pObj->m_ClipPath.SetNull();
+ m_Size = safeSize.ValueOrDie();
+ m_pData.Reset(
+ std::unique_ptr<uint8_t, FxFreeDeleter>(FX_Alloc(uint8_t, m_Size)));
+ uint32_t pos = 0;
+ for (const auto& stream : m_StreamArray) {
+ memcpy(m_pData.Get() + pos, stream->GetData(), stream->GetSize());
+ pos += stream->GetSize();
+ m_pData.Get()[pos++] = ' ';
}
- m_bIsDone = true;
- return false;
+ m_StreamArray.clear();
+ } else {
+ m_pData.Reset(m_pSingleStream->GetData());
+ m_Size = m_pSingleStream->GetSize();
}
- if (pPause && pPause->NeedToPauseNow())
- break;
+ m_InternalStage = STAGE_PARSE;
+ m_CurrentOffset = 0;
+ return;
+ }
+
+ CPDF_Array* pContent =
+ m_pObjectHolder->GetFormDict()->GetArrayFor("Contents");
+ CPDF_Stream* pStreamObj = ToStream(
+ pContent ? pContent->GetDirectObjectAt(m_CurrentOffset) : nullptr);
+ m_StreamArray[m_CurrentOffset] =
+ pdfium::MakeRetain<CPDF_StreamAcc>(pStreamObj);
+ m_StreamArray[m_CurrentOffset]->LoadAllDataFiltered();
+ m_CurrentOffset++;
+}
+
+void CPDF_ContentParser::StageParse() {
+ if (!m_pParser) {
+ m_parsedSet = pdfium::MakeUnique<std::set<const uint8_t*>>();
+ m_pParser = pdfium::MakeUnique<CPDF_StreamContentParser>(
+ m_pObjectHolder->GetDocument(), m_pObjectHolder->m_pPageResources.Get(),
+ nullptr, nullptr, m_pObjectHolder.Get(),
+ m_pObjectHolder->m_pResources.Get(), m_pObjectHolder->GetBBox(),
+ nullptr, m_parsedSet.get());
+ m_pParser->GetCurStates()->m_ColorState.SetDefault();
+ }
+ if (m_CurrentOffset >= m_Size) {
+ m_InternalStage = STAGE_CHECKCLIP;
+ return;
+ }
+
+ m_CurrentOffset +=
+ m_pParser->Parse(m_pData.Get() + m_CurrentOffset,
+ m_Size - m_CurrentOffset, PARSE_STEP_LIMIT);
+}
+
+void CPDF_ContentParser::StageCheckClip() {
+ if (m_pType3Char) {
+ m_pType3Char->InitializeFromStreamData(m_pParser->IsColored(),
+ m_pParser->GetType3Data());
+ }
+
+ for (auto& pObj : *m_pObjectHolder->GetPageObjectList()) {
+ if (!pObj->m_ClipPath.HasRef())
+ continue;
+ if (pObj->m_ClipPath.GetPathCount() != 1)
+ continue;
+ if (pObj->m_ClipPath.GetTextCount() > 0)
+ continue;
+
+ CPDF_Path ClipPath = pObj->m_ClipPath.GetPath(0);
+ if (!ClipPath.IsRect() || pObj->IsShading())
+ continue;
+
+ CFX_PointF point0 = ClipPath.GetPoint(0);
+ CFX_PointF point2 = ClipPath.GetPoint(2);
+ CFX_FloatRect old_rect(point0.x, point0.y, point2.x, point2.y);
+ CFX_FloatRect obj_rect(pObj->m_Left, pObj->m_Bottom, pObj->m_Right,
+ pObj->m_Top);
+ if (old_rect.Contains(obj_rect))
+ pObj->m_ClipPath.SetNull();
}
- return true;
+ m_bIsDone = true;
}
diff --git a/core/fpdfapi/page/cpdf_contentparser.h b/core/fpdfapi/page/cpdf_contentparser.h
index b66783775b..d530a1407e 100644
--- a/core/fpdfapi/page/cpdf_contentparser.h
+++ b/core/fpdfapi/page/cpdf_contentparser.h
@@ -46,6 +46,10 @@ class CPDF_ContentParser {
STAGE_CHECKCLIP,
};
+ void StageGetContent();
+ void StageParse();
+ void StageCheckClip();
+
bool m_bIsDone = false;
InternalStage m_InternalStage;
UnownedPtr<CPDF_PageObjectHolder> const m_pObjectHolder;