diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-13 21:27:44 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-13 21:27:44 +0000 |
| commit | 9c112f92d4c2046d5a4f8538f4d18b74a87649d4 (patch) | |
| tree | ffd0ba656902d28257e6560c8713df895802d313 /xfa/fxfa/parser/cxfa_document_parser.cpp | |
| parent | 20eb52e66c565d36a78dc1399f04f7485ccf8fe6 (diff) | |
| download | pdfium-9c112f92d4c2046d5a4f8538f4d18b74a87649d4.tar.xz | |
Cleanup CFX_XMLDoc::SaveXMLNode
The CFX_XMLDoc::SaveXMLNode method is almost an exact copy of the
CFX_XMLNode::SaveXMLNode. This CL removes the XMLDoc variant and calls
the XMLNode method directly.
This Removes the need to pass the CXFA_DocumentParser into CXFA_Document
and we can instead pass in the CXFA_FFNotify object directly.
Change-Id: Ic3c8c66375483fe73b44dd84064a1b71b039d61c
Reviewed-on: https://pdfium-review.googlesource.com/26530
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_document_parser.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_document_parser.cpp | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/xfa/fxfa/parser/cxfa_document_parser.cpp b/xfa/fxfa/parser/cxfa_document_parser.cpp index 6446ea31ce..652694e214 100644 --- a/xfa/fxfa/parser/cxfa_document_parser.cpp +++ b/xfa/fxfa/parser/cxfa_document_parser.cpp @@ -25,7 +25,7 @@ int32_t CXFA_DocumentParser::StartParse( int32_t nRetStatus = m_nodeParser.StartParse(pStream, ePacketID); if (nRetStatus == XFA_PARSESTATUS_Ready) { - m_pDocument = pdfium::MakeUnique<CXFA_Document>(this); + m_pDocument = pdfium::MakeUnique<CXFA_Document>(GetNotify()); m_nodeParser.SetFactory(m_pDocument.get()); } return nRetStatus; @@ -40,10 +40,6 @@ int32_t CXFA_DocumentParser::DoParse() { return nRetStatus; } -CFX_XMLDoc* CXFA_DocumentParser::GetXMLDoc() const { - return m_nodeParser.GetXMLDoc(); -} - CXFA_FFNotify* CXFA_DocumentParser::GetNotify() const { return m_pNotify.Get(); } |