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_dataimporter.cpp | |
| 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_dataimporter.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_dataimporter.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/xfa/fxfa/parser/cxfa_dataimporter.cpp b/xfa/fxfa/parser/cxfa_dataimporter.cpp index 01e95b1c3a..b44e5e0552 100644 --- a/xfa/fxfa/parser/cxfa_dataimporter.cpp +++ b/xfa/fxfa/parser/cxfa_dataimporter.cpp @@ -27,8 +27,7 @@ CXFA_DataImporter::~CXFA_DataImporter() {} bool CXFA_DataImporter::ImportData( const RetainPtr<IFX_SeekableStream>& pDataDocument) { CXFA_SimpleParser parser(m_pDocument.Get()); - int status = parser.Parse(pDataDocument, XFA_PacketType::Datasets); - if (status < XFA_PARSESTATUS_Done) + if (!parser.Parse(pDataDocument, XFA_PacketType::Datasets)) return false; CXFA_Node* pImportDataRoot = parser.GetRootNode(); |