diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-03-22 13:01:10 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-03-22 20:25:33 +0000 |
| commit | 3ae3033bff2582029984f91f40f4a8b748a30c96 (patch) | |
| tree | d2f22825c4eb2f0f54492536e1f7db1f9222bcb0 /xfa/fde/xml/fde_xml_imp.h | |
| parent | 2fbfb84aedccefbd1c1a7bb2c5b2a723b3f3dd77 (diff) | |
| download | pdfium-3ae3033bff2582029984f91f40f4a8b748a30c96.tar.xz | |
Use unique_ptr for CFDE_XMLDoc::m_pRoot
Remove some less helpful helper methods.
Change-Id: Id027e31b6584156263c4aa0c2888e91fa44e4b0b
Reviewed-on: https://pdfium-review.googlesource.com/3150
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fde/xml/fde_xml_imp.h')
| -rw-r--r-- | xfa/fde/xml/fde_xml_imp.h | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/xfa/fde/xml/fde_xml_imp.h b/xfa/fde/xml/fde_xml_imp.h index 687a0cd416..ad2ccc467c 100644 --- a/xfa/fde/xml/fde_xml_imp.h +++ b/xfa/fde/xml/fde_xml_imp.h @@ -194,19 +194,16 @@ class CFDE_XMLDoc { bool LoadXML(std::unique_ptr<IFDE_XMLParser> pXMLParser); int32_t DoLoad(IFX_Pause* pPause = nullptr); void CloseXML(); - CFDE_XMLNode* GetRoot() const { return m_pRoot; } + CFDE_XMLNode* GetRoot() const { return m_pRoot.get(); } void SaveXML(CFX_RetainPtr<IFGAS_Stream>& pXMLStream, bool bSaveBOM = true); void SaveXMLNode(const CFX_RetainPtr<IFGAS_Stream>& pXMLStream, CFDE_XMLNode* pNode); - protected: - void Reset(bool bInitRoot); - void ReleaseParser(); - - CFX_RetainPtr<IFGAS_Stream> m_pStream; + private: int32_t m_iStatus; - CFDE_XMLNode* m_pRoot; + std::unique_ptr<CFDE_XMLNode> m_pRoot; std::unique_ptr<IFDE_XMLParser> m_pXMLParser; + CFX_RetainPtr<IFGAS_Stream> m_pStream; }; class IFDE_XMLParser { |