summaryrefslogtreecommitdiff
path: root/fpdfsdk/src
diff options
context:
space:
mode:
authorOliver Chang <ochang@chromium.org>2015-10-30 15:28:47 -0700
committerOliver Chang <ochang@chromium.org>2015-10-30 15:28:47 -0700
commit34c509e0e0fbd64d4380f130c48434b07f60b69e (patch)
treeaddd5543aae6edbc266fa93d43e89f162b2f5a77 /fpdfsdk/src
parent81edc51ef8242205c5a1192bc33bf16d8c96120b (diff)
downloadpdfium-34c509e0e0fbd64d4380f130c48434b07f60b69e.tar.xz
Convert a bunch of raw pointers in fsdk_mgr to unique_ptrs.
R=tsepez@chromium.org, thestig@chromium.org Review URL: https://codereview.chromium.org/1425233002 .
Diffstat (limited to 'fpdfsdk/src')
-rw-r--r--fpdfsdk/src/fsdk_mgr.cpp52
1 files changed, 13 insertions, 39 deletions
diff --git a/fpdfsdk/src/fsdk_mgr.cpp b/fpdfsdk/src/fsdk_mgr.cpp
index 60fd0eff0b..841bd197e0 100644
--- a/fpdfsdk/src/fsdk_mgr.cpp
+++ b/fpdfsdk/src/fsdk_mgr.cpp
@@ -210,27 +210,13 @@ FX_SYSTEMTIME CFX_SystemHandler::GetLocalTime() {
CPDFDoc_Environment::CPDFDoc_Environment(CPDF_Document* pDoc,
FPDF_FORMFILLINFO* pFFinfo)
- : m_pAnnotHandlerMgr(NULL),
- m_pActionHandler(NULL),
- m_pInfo(pFFinfo),
+ : m_pInfo(pFFinfo),
m_pSDKDoc(NULL),
m_pPDFDoc(pDoc),
- m_pIFormFiller(NULL) {
- m_pSysHandler = new CFX_SystemHandler(this);
+ m_pSysHandler(new CFX_SystemHandler(this)) {
}
CPDFDoc_Environment::~CPDFDoc_Environment() {
- delete m_pIFormFiller;
- m_pIFormFiller = NULL;
-
- delete m_pSysHandler;
- m_pSysHandler = NULL;
-
- delete m_pAnnotHandlerMgr;
- m_pAnnotHandlerMgr = NULL;
-
- delete m_pActionHandler;
- m_pActionHandler = NULL;
}
int CPDFDoc_Environment::JS_appAlert(const FX_WCHAR* Msg,
@@ -385,31 +371,25 @@ IJS_Runtime* CPDFDoc_Environment::GetJSRuntime() {
CPDFSDK_AnnotHandlerMgr* CPDFDoc_Environment::GetAnnotHandlerMgr() {
if (!m_pAnnotHandlerMgr)
- m_pAnnotHandlerMgr = new CPDFSDK_AnnotHandlerMgr(this);
- return m_pAnnotHandlerMgr;
+ m_pAnnotHandlerMgr.reset(new CPDFSDK_AnnotHandlerMgr(this));
+ return m_pAnnotHandlerMgr.get();
}
CPDFSDK_ActionHandler* CPDFDoc_Environment::GetActionHander() {
if (!m_pActionHandler)
- m_pActionHandler = new CPDFSDK_ActionHandler();
- return m_pActionHandler;
+ m_pActionHandler.reset(new CPDFSDK_ActionHandler());
+ return m_pActionHandler.get();
}
CFFL_IFormFiller* CPDFDoc_Environment::GetIFormFiller() {
if (!m_pIFormFiller)
- m_pIFormFiller = new CFFL_IFormFiller(this);
- return m_pIFormFiller;
+ m_pIFormFiller.reset(new CFFL_IFormFiller(this));
+ return m_pIFormFiller.get();
}
CPDFSDK_Document::CPDFSDK_Document(CPDF_Document* pDoc,
CPDFDoc_Environment* pEnv)
- : m_pDoc(pDoc),
- m_pInterForm(nullptr),
- m_pFocusAnnot(nullptr),
- m_pEnv(pEnv),
- m_pOccontent(nullptr),
- m_bChangeMask(FALSE),
- m_bBeingDestroyed(FALSE) {
+ : m_pDoc(pDoc), m_pFocusAnnot(nullptr), m_pEnv(pEnv), m_bChangeMask(FALSE) {
}
CPDFSDK_Document::~CPDFSDK_Document() {
@@ -421,12 +401,6 @@ CPDFSDK_Document::~CPDFSDK_Document() {
for (auto& it : m_pageMap)
delete it.second;
m_pageMap.clear();
-
- delete m_pInterForm;
- m_pInterForm = nullptr;
-
- delete m_pOccontent;
- m_pOccontent = nullptr;
}
CPDFSDK_PageView* CPDFSDK_Document::GetPageView(CPDF_Page* pPDFPage,
@@ -503,8 +477,8 @@ FX_BOOL CPDFSDK_Document::ProcOpenAction() {
CPDF_OCContext* CPDFSDK_Document::GetOCContext() {
if (!m_pOccontent)
- m_pOccontent = new CPDF_OCContext(m_pDoc);
- return m_pOccontent;
+ m_pOccontent.reset(new CPDF_OCContext(m_pDoc));
+ return m_pOccontent.get();
}
void CPDFSDK_Document::ReMovePageView(CPDF_Page* pPDFPage) {
@@ -530,8 +504,8 @@ CPDF_Page* CPDFSDK_Document::GetPage(int nIndex) {
CPDFSDK_InterForm* CPDFSDK_Document::GetInterForm() {
if (!m_pInterForm)
- m_pInterForm = new CPDFSDK_InterForm(this);
- return m_pInterForm;
+ m_pInterForm.reset(new CPDFSDK_InterForm(this));
+ return m_pInterForm.get();
}
void CPDFSDK_Document::UpdateAllViews(CPDFSDK_PageView* pSender,