diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-15 18:12:29 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-15 18:12:29 +0000 |
| commit | e40678ed8a22ecd57421877af39cf7f281f618c4 (patch) | |
| tree | 50bbf114829ad3b4852b3739f65c6385091c4ba0 /xfa/fxfa/parser/cxfa_document_parser.h | |
| parent | 625e6fec9ddd1d023116f7bd22d2192ef9bc49c8 (diff) | |
| download | pdfium-e40678ed8a22ecd57421877af39cf7f281f618c4.tar.xz | |
Make the CFX_XMLNode a MaybeOwned pointer
This CL removes the HasOwnXML flag from the CXFA_Node objects and
instead uses a MaybeOwned pointer to keep track of the XML nodes.
Change-Id: Ie678258247ec21ecb15c639647b189e140586d25
Reviewed-on: https://pdfium-review.googlesource.com/26811
Reviewed-by: Tom Sepez <tsepez@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 | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/xfa/fxfa/parser/cxfa_document_parser.h b/xfa/fxfa/parser/cxfa_document_parser.h index c245843c0f..77265e4d05 100644 --- a/xfa/fxfa/parser/cxfa_document_parser.h +++ b/xfa/fxfa/parser/cxfa_document_parser.h @@ -31,6 +31,8 @@ class CXFA_DocumentParser { 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<CXFA_Document> m_pDocument; CXFA_SimpleParser m_nodeParser; }; |