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.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.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_document.cpp | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/xfa/fxfa/parser/cxfa_document.cpp b/xfa/fxfa/parser/cxfa_document.cpp index a21d3f974c..2a9d80e0cd 100644 --- a/xfa/fxfa/parser/cxfa_document.cpp +++ b/xfa/fxfa/parser/cxfa_document.cpp @@ -88,14 +88,12 @@ void MergeNode(CXFA_Document* pDocument, } // namespace -CXFA_Document::CXFA_Document(CXFA_DocumentParser* pParser) +CXFA_Document::CXFA_Document(CXFA_FFNotify* notify) : CXFA_NodeOwner(), - m_pParser(pParser), + notify_(notify), m_pRootNode(nullptr), m_eCurVersionMode(XFA_VERSION_DEFAULT), - m_dwDocFlags(0) { - ASSERT(m_pParser); -} + m_dwDocFlags(0) {} CXFA_Document::~CXFA_Document() { // Remove all the bindings before freeing the node as the ownership is wonky. @@ -125,14 +123,6 @@ void CXFA_Document::ClearLayoutData() { m_pScriptSignature.reset(); } -CFX_XMLDoc* CXFA_Document::GetXMLDoc() const { - return m_pParser->GetXMLDoc(); -} - -CXFA_FFNotify* CXFA_Document::GetNotify() const { - return m_pParser->GetNotify(); -} - CXFA_Object* CXFA_Document::GetXFAObject(XFA_HashCode dwNodeNameHash) { switch (dwNodeNameHash) { case XFA_HASHCODE_Data: { |