summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArtem Strygin <art-snake@yandex-team.ru>2017-10-30 21:03:02 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-10-30 21:03:02 +0000
commitcaee9596e926a41a23fa0ddebd10d509db4b23ab (patch)
tree9864d853d3a8f4a9a506525781f9dfe858ad63e5
parent5c02f12cfb3b6a9c27085a48093956f730762534 (diff)
downloadpdfium-caee9596e926a41a23fa0ddebd10d509db4b23ab.tar.xz
Remove useless code.
Change-Id: I5fa9472eecc176d25213fd446a4abc06e815a48b Reviewed-on: https://pdfium-review.googlesource.com/15592 Commit-Queue: Art Snake <art-snake@yandex-team.ru> Reviewed-by: dsinclair <dsinclair@chromium.org>
-rw-r--r--core/fpdfapi/parser/cpdf_data_avail.cpp110
-rw-r--r--core/fpdfapi/parser/cpdf_data_avail.h7
2 files changed, 0 insertions, 117 deletions
diff --git a/core/fpdfapi/parser/cpdf_data_avail.cpp b/core/fpdfapi/parser/cpdf_data_avail.cpp
index 549ed75425..93dd39b87b 100644
--- a/core/fpdfapi/parser/cpdf_data_avail.cpp
+++ b/core/fpdfapi/parser/cpdf_data_avail.cpp
@@ -118,78 +118,6 @@ void CPDF_DataAvail::SetDocument(CPDF_Document* pDoc) {
m_pDocument = pDoc;
}
-bool CPDF_DataAvail::AreObjectsAvailable(std::vector<CPDF_Object*>& obj_array,
- bool bParsePage,
- std::vector<CPDF_Object*>& ret_array) {
- if (obj_array.empty())
- return true;
-
- uint32_t count = 0;
- std::vector<CPDF_Object*> new_obj_array;
- for (CPDF_Object* pObj : obj_array) {
- if (!pObj)
- continue;
-
- int32_t type = pObj->GetType();
- switch (type) {
- case CPDF_Object::ARRAY: {
- CPDF_Array* pArray = pObj->AsArray();
- for (size_t k = 0; k < pArray->GetCount(); ++k)
- new_obj_array.push_back(pArray->GetObjectAt(k));
- break;
- }
- case CPDF_Object::STREAM:
- pObj = pObj->GetDict();
- case CPDF_Object::DICTIONARY: {
- CPDF_Dictionary* pDict = pObj->GetDict();
- if (pDict && pDict->GetStringFor("Type") == "Page" && !bParsePage)
- continue;
-
- for (const auto& it : *pDict) {
- if (it.first != "Parent")
- new_obj_array.push_back(it.second.get());
- }
- break;
- }
- case CPDF_Object::REFERENCE: {
- const CPDF_ReadValidator::Session read_session(GetValidator().Get());
-
- CPDF_Reference* pRef = pObj->AsReference();
- const uint32_t dwNum = pRef->GetRefObjNum();
-
- if (pdfium::ContainsKey(m_ObjectSet, dwNum))
- break;
-
- CPDF_Object* pReferred = m_pDocument->GetOrParseIndirectObject(dwNum);
- if (GetValidator()->has_read_problems()) {
- ASSERT(!pReferred);
- ret_array.push_back(pObj);
- ++count;
- break;
- }
- m_ObjectSet.insert(dwNum);
- if (pReferred)
- new_obj_array.push_back(pReferred);
-
- break;
- }
- }
- }
-
- if (count > 0) {
- for (CPDF_Object* pObj : new_obj_array) {
- CPDF_Reference* pRef = pObj->AsReference();
- if (pRef && pdfium::ContainsKey(m_ObjectSet, pRef->GetRefObjNum()))
- continue;
- ret_array.push_back(pObj);
- }
- return false;
- }
-
- obj_array = new_obj_array;
- return AreObjectsAvailable(obj_array, false, ret_array);
-}
-
CPDF_DataAvail::DocAvailStatus CPDF_DataAvail::IsDocAvail(
DownloadHints* pHints) {
if (!m_dwFileLen)
@@ -1137,44 +1065,6 @@ CPDF_DataAvail::DocAvailStatus CPDF_DataAvail::CheckLinearizedData() {
return m_bLinearedDataOK ? DataAvailable : DataNotAvailable;
}
-bool CPDF_DataAvail::CheckPageAnnots(uint32_t dwPage) {
- if (m_objs_array.empty()) {
- m_ObjectSet.clear();
-
- FX_SAFE_INT32 safePage = pdfium::base::checked_cast<int32_t>(dwPage);
- CPDF_Dictionary* pPageDict = m_pDocument->GetPage(safePage.ValueOrDie());
- if (!pPageDict)
- return true;
-
- CPDF_Object* pAnnots = pPageDict->GetObjectFor("Annots");
- if (!pAnnots)
- return true;
-
- std::vector<CPDF_Object*> obj_array;
- obj_array.push_back(pAnnots);
- if (!AreObjectsAvailable(obj_array, false, m_objs_array))
- return false;
-
- m_objs_array.clear();
- return true;
- }
-
- std::vector<CPDF_Object*> new_objs_array;
- if (!AreObjectsAvailable(m_objs_array, false, new_objs_array)) {
- m_objs_array = new_objs_array;
- return false;
- }
- m_objs_array.clear();
- return true;
-}
-
-CPDF_DataAvail::DocAvailStatus CPDF_DataAvail::CheckLinearizedFirstPage(
- uint32_t dwPage) {
- if (!ValidatePage(dwPage))
- return DataError;
- return DataAvailable;
-}
-
CPDF_DataAvail::DocAvailStatus CPDF_DataAvail::IsPageAvail(
uint32_t dwPage,
DownloadHints* pHints) {
diff --git a/core/fpdfapi/parser/cpdf_data_avail.h b/core/fpdfapi/parser/cpdf_data_avail.h
index a887d94901..2d46be1152 100644
--- a/core/fpdfapi/parser/cpdf_data_avail.h
+++ b/core/fpdfapi/parser/cpdf_data_avail.h
@@ -124,9 +124,6 @@ class CPDF_DataAvail final {
static const int kMaxPageRecursionDepth = 1024;
- bool AreObjectsAvailable(std::vector<CPDF_Object*>& obj_array,
- bool bParsePage,
- std::vector<CPDF_Object*>& ret_array);
bool CheckDocStatus();
bool CheckHeader();
bool CheckFirstPage();
@@ -159,9 +156,7 @@ class CPDF_DataAvail final {
bool LoadAllXref();
bool LoadAllFile();
DocAvailStatus CheckLinearizedData();
- bool CheckPageAnnots(uint32_t dwPage);
- DocAvailStatus CheckLinearizedFirstPage(uint32_t dwPage);
bool CheckPage(uint32_t dwPage);
bool LoadDocPages();
bool LoadDocPage(uint32_t dwPage);
@@ -194,8 +189,6 @@ class CPDF_DataAvail final {
PDF_DATAAVAIL_STATUS m_docStatus;
FX_FILESIZE m_dwFileLen;
CPDF_Document* m_pDocument;
- std::set<uint32_t> m_ObjectSet;
- std::vector<CPDF_Object*> m_objs_array;
FX_FILESIZE m_Pos;
FX_FILESIZE m_bufferOffset;
uint32_t m_bufferSize;