diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-04-12 13:21:48 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-12 13:21:48 +0000 |
| commit | 2efb11c89ee200473f5675ba3d3a946d5606e813 (patch) | |
| tree | 52b46b0ef3ee3bc91046b217c65aa13e5dd9a06d /xfa/fxfa/parser/cxfa_document_parser.h | |
| parent | a995d6fd9b862dbd37aebb9c323766bb5d11d389 (diff) | |
| download | pdfium-2efb11c89ee200473f5675ba3d3a946d5606e813.tar.xz | |
Retrieve the XML tree in the DocumentParser
This CL changes the document parser to store the parsed XML tree. That
way we no longer need to store the pointer to the CXFA_SimpleParser in
the CXFA_DocumentParser.
Change-Id: I9272150e73613411cb1bb59c3cf77c7af6cc799d
Reviewed-on: https://pdfium-review.googlesource.com/30211
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_document_parser.h')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_document_parser.h | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/xfa/fxfa/parser/cxfa_document_parser.h b/xfa/fxfa/parser/cxfa_document_parser.h index c7056c215a..1075f48ef4 100644 --- a/xfa/fxfa/parser/cxfa_document_parser.h +++ b/xfa/fxfa/parser/cxfa_document_parser.h @@ -22,18 +22,16 @@ class CXFA_DocumentParser { explicit CXFA_DocumentParser(CXFA_FFNotify* pNotify); ~CXFA_DocumentParser(); - int32_t Parse(const RetainPtr<IFX_SeekableStream>& pStream, - XFA_PacketType ePacketID); + bool Parse(const RetainPtr<IFX_SeekableStream>& pStream, + XFA_PacketType ePacketID); CXFA_FFNotify* GetNotify() const; CXFA_Document* GetDocument() const; private: UnownedPtr<CXFA_FFNotify> const m_pNotify; - // Note, the |m_nodeParser| has an unowned pointer to the |m_pDocument| so - // the |m_nodeParser| must be cleaned up first. + std::unique_ptr<CFX_XMLNode> m_pXMLRoot; std::unique_ptr<CXFA_Document> m_pDocument; - CXFA_SimpleParser m_nodeParser; }; #endif // XFA_FXFA_PARSER_CXFA_DOCUMENT_PARSER_H_ |