summaryrefslogtreecommitdiff
path: root/core/fpdfdoc
diff options
context:
space:
mode:
Diffstat (limited to 'core/fpdfdoc')
-rw-r--r--core/fpdfdoc/cpdf_nametree.cpp12
-rw-r--r--core/fpdfdoc/cpdf_nametree.h8
-rw-r--r--core/fpdfdoc/cpdf_structelement.h3
-rw-r--r--core/fpdfdoc/cpdf_structtree.h13
-rw-r--r--core/fpdfdoc/cpvt_fontmap.cpp16
-rw-r--r--core/fpdfdoc/cpvt_fontmap.h9
6 files changed, 34 insertions, 27 deletions
diff --git a/core/fpdfdoc/cpdf_nametree.cpp b/core/fpdfdoc/cpdf_nametree.cpp
index 5ecdb4863b..b3808bddba 100644
--- a/core/fpdfdoc/cpdf_nametree.cpp
+++ b/core/fpdfdoc/cpdf_nametree.cpp
@@ -136,6 +136,8 @@ size_t CountNames(CPDF_Dictionary* pNode, int nLevel = 0) {
} // namespace
+CPDF_NameTree::CPDF_NameTree(CPDF_Dictionary* pRoot) : m_pRoot(pRoot) {}
+
CPDF_NameTree::CPDF_NameTree(CPDF_Document* pDoc,
const CFX_ByteString& category)
: m_pRoot(nullptr) {
@@ -150,8 +152,10 @@ CPDF_NameTree::CPDF_NameTree(CPDF_Document* pDoc,
m_pRoot = pNames->GetDictFor(category);
}
+CPDF_NameTree::~CPDF_NameTree() {}
+
size_t CPDF_NameTree::GetCount() const {
- return m_pRoot ? ::CountNames(m_pRoot) : 0;
+ return m_pRoot ? ::CountNames(m_pRoot.Get()) : 0;
}
int CPDF_NameTree::GetIndex(const CFX_ByteString& csName) const {
@@ -159,7 +163,7 @@ int CPDF_NameTree::GetIndex(const CFX_ByteString& csName) const {
return -1;
size_t nIndex = 0;
- if (!SearchNameNode(m_pRoot, csName, nIndex, nullptr))
+ if (!SearchNameNode(m_pRoot.Get(), csName, nIndex, nullptr))
return -1;
return nIndex;
}
@@ -171,7 +175,7 @@ CPDF_Object* CPDF_NameTree::LookupValueAndName(int nIndex,
return nullptr;
size_t nCurIndex = 0;
- return SearchNameNode(m_pRoot, nIndex, nCurIndex, csName, nullptr);
+ return SearchNameNode(m_pRoot.Get(), nIndex, nCurIndex, csName, nullptr);
}
CPDF_Object* CPDF_NameTree::LookupValue(const CFX_ByteString& csName) const {
@@ -179,7 +183,7 @@ CPDF_Object* CPDF_NameTree::LookupValue(const CFX_ByteString& csName) const {
return nullptr;
size_t nIndex = 0;
- return SearchNameNode(m_pRoot, csName, nIndex, nullptr);
+ return SearchNameNode(m_pRoot.Get(), csName, nIndex, nullptr);
}
CPDF_Array* CPDF_NameTree::LookupNamedDest(CPDF_Document* pDoc,
diff --git a/core/fpdfdoc/cpdf_nametree.h b/core/fpdfdoc/cpdf_nametree.h
index 7a792aef59..69000f32b2 100644
--- a/core/fpdfdoc/cpdf_nametree.h
+++ b/core/fpdfdoc/cpdf_nametree.h
@@ -7,6 +7,7 @@
#ifndef CORE_FPDFDOC_CPDF_NAMETREE_H_
#define CORE_FPDFDOC_CPDF_NAMETREE_H_
+#include "core/fxcrt/cfx_unowned_ptr.h"
#include "core/fxcrt/fx_string.h"
class CPDF_Array;
@@ -16,8 +17,9 @@ class CPDF_Object;
class CPDF_NameTree {
public:
- explicit CPDF_NameTree(CPDF_Dictionary* pRoot) : m_pRoot(pRoot) {}
+ explicit CPDF_NameTree(CPDF_Dictionary* pRoot);
CPDF_NameTree(CPDF_Document* pDoc, const CFX_ByteString& category);
+ ~CPDF_NameTree();
CPDF_Object* LookupValueAndName(int nIndex, CFX_ByteString* csName) const;
CPDF_Object* LookupValue(const CFX_ByteString& csName) const;
@@ -25,10 +27,10 @@ class CPDF_NameTree {
int GetIndex(const CFX_ByteString& csName) const;
size_t GetCount() const;
- CPDF_Dictionary* GetRoot() const { return m_pRoot; }
+ CPDF_Dictionary* GetRoot() const { return m_pRoot.Get(); }
private:
- CPDF_Dictionary* m_pRoot;
+ CFX_UnownedPtr<CPDF_Dictionary> m_pRoot;
};
#endif // CORE_FPDFDOC_CPDF_NAMETREE_H_
diff --git a/core/fpdfdoc/cpdf_structelement.h b/core/fpdfdoc/cpdf_structelement.h
index c65363db53..d8820a141c 100644
--- a/core/fpdfdoc/cpdf_structelement.h
+++ b/core/fpdfdoc/cpdf_structelement.h
@@ -10,6 +10,7 @@
#include <vector>
#include "core/fxcrt/cfx_retain_ptr.h"
+#include "core/fxcrt/cfx_unowned_ptr.h"
#include "core/fxcrt/fx_string.h"
#include "core/fxge/fx_dib.h"
@@ -27,7 +28,7 @@ class CPDF_StructKid {
enum { Invalid, Element, PageContent, StreamContent, Object } m_Type;
CFX_RetainPtr<CPDF_StructElement> m_pElement; // For Element.
- CPDF_Dictionary* m_pDict; // For Element.
+ CFX_UnownedPtr<CPDF_Dictionary> m_pDict; // For Element.
uint32_t m_PageObjNum; // For PageContent, StreamContent, Object.
uint32_t m_RefObjNum; // For StreamContent, Object.
uint32_t m_ContentId; // For PageContent, StreamContent.
diff --git a/core/fpdfdoc/cpdf_structtree.h b/core/fpdfdoc/cpdf_structtree.h
index aa30f802db..037512acfc 100644
--- a/core/fpdfdoc/cpdf_structtree.h
+++ b/core/fpdfdoc/cpdf_structtree.h
@@ -12,6 +12,7 @@
#include <vector>
#include "core/fxcrt/cfx_retain_ptr.h"
+#include "core/fxcrt/cfx_unowned_ptr.h"
class CPDF_Dictionary;
class CPDF_Document;
@@ -28,9 +29,9 @@ class CPDF_StructTree {
int CountTopElements() const;
CPDF_StructElement* GetTopElement(int i) const;
- const CPDF_Dictionary* GetRoleMap() const { return m_pRoleMap; }
- const CPDF_Dictionary* GetPage() const { return m_pPage; }
- const CPDF_Dictionary* GetTreeRoot() const { return m_pTreeRoot; }
+ const CPDF_Dictionary* GetRoleMap() const { return m_pRoleMap.Get(); }
+ const CPDF_Dictionary* GetPage() const { return m_pPage.Get(); }
+ const CPDF_Dictionary* GetTreeRoot() const { return m_pTreeRoot.Get(); }
private:
void LoadPageTree(const CPDF_Dictionary* pPageDict);
@@ -41,9 +42,9 @@ class CPDF_StructTree {
bool AddTopLevelNode(CPDF_Dictionary* pDict,
const CFX_RetainPtr<CPDF_StructElement>& pElement);
- const CPDF_Dictionary* const m_pTreeRoot;
- const CPDF_Dictionary* const m_pRoleMap;
- const CPDF_Dictionary* m_pPage;
+ CFX_UnownedPtr<const CPDF_Dictionary> const m_pTreeRoot;
+ CFX_UnownedPtr<const CPDF_Dictionary> const m_pRoleMap;
+ CFX_UnownedPtr<const CPDF_Dictionary> m_pPage;
std::vector<CFX_RetainPtr<CPDF_StructElement>> m_Kids;
};
diff --git a/core/fpdfdoc/cpvt_fontmap.cpp b/core/fpdfdoc/cpvt_fontmap.cpp
index c6e1a85dae..0edabd1bc3 100644
--- a/core/fpdfdoc/cpvt_fontmap.cpp
+++ b/core/fpdfdoc/cpvt_fontmap.cpp
@@ -21,9 +21,7 @@ CPVT_FontMap::CPVT_FontMap(CPDF_Document* pDoc,
: m_pDocument(pDoc),
m_pResDict(pResDict),
m_pDefFont(pDefFont),
- m_sDefFontAlias(sDefFontAlias),
- m_pSysFont(nullptr),
- m_sSysFontAlias() {}
+ m_sDefFontAlias(sDefFontAlias) {}
CPVT_FontMap::~CPVT_FontMap() {}
@@ -49,13 +47,13 @@ CPDF_Font* CPVT_FontMap::GetAnnotSysPDFFont(CPDF_Document* pDoc,
CPDF_Font* CPVT_FontMap::GetPDFFont(int32_t nFontIndex) {
switch (nFontIndex) {
case 0:
- return m_pDefFont;
+ return m_pDefFont.Get();
case 1:
if (!m_pSysFont) {
- m_pSysFont =
- GetAnnotSysPDFFont(m_pDocument, m_pResDict, &m_sSysFontAlias);
+ m_pSysFont = GetAnnotSysPDFFont(m_pDocument.Get(), m_pResDict.Get(),
+ &m_sSysFontAlias);
}
- return m_pSysFont;
+ return m_pSysFont.Get();
default:
return nullptr;
}
@@ -67,8 +65,8 @@ CFX_ByteString CPVT_FontMap::GetPDFFontAlias(int32_t nFontIndex) {
return m_sDefFontAlias;
case 1:
if (!m_pSysFont) {
- m_pSysFont =
- GetAnnotSysPDFFont(m_pDocument, m_pResDict, &m_sSysFontAlias);
+ m_pSysFont = GetAnnotSysPDFFont(m_pDocument.Get(), m_pResDict.Get(),
+ &m_sSysFontAlias);
}
return m_sSysFontAlias;
default:
diff --git a/core/fpdfdoc/cpvt_fontmap.h b/core/fpdfdoc/cpvt_fontmap.h
index 6080010cac..da69463d95 100644
--- a/core/fpdfdoc/cpvt_fontmap.h
+++ b/core/fpdfdoc/cpvt_fontmap.h
@@ -10,6 +10,7 @@
#include <stdint.h>
#include "core/fpdfdoc/ipvt_fontmap.h"
+#include "core/fxcrt/cfx_unowned_ptr.h"
#include "core/fxcrt/fx_string.h"
class CPDF_Document;
@@ -38,11 +39,11 @@ class CPVT_FontMap : public IPVT_FontMap {
CFX_ByteString* sSysFontAlias);
private:
- CPDF_Document* const m_pDocument;
- const CPDF_Dictionary* const m_pResDict;
- CPDF_Font* const m_pDefFont;
+ CFX_UnownedPtr<CPDF_Document> const m_pDocument;
+ CFX_UnownedPtr<CPDF_Dictionary> const m_pResDict;
+ CFX_UnownedPtr<CPDF_Font> const m_pDefFont;
+ CFX_UnownedPtr<CPDF_Font> m_pSysFont;
const CFX_ByteString m_sDefFontAlias;
- CPDF_Font* m_pSysFont;
CFX_ByteString m_sSysFontAlias;
};