diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-13 20:55:03 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-13 20:55:03 +0000 |
commit | 80bf582f00adeefae67ae432b74c3b0609e7845e (patch) | |
tree | 18c88bc620fcded6fc39cdfe3a5b0ae04643424d /xfa/fxfa/parser/cxfa_nodeowner.cpp | |
parent | 566d0b9486f024f253d1c331866b04529c9a22b3 (diff) | |
download | pdfium-80bf582f00adeefae67ae432b74c3b0609e7845e.tar.xz |
Cleanup CXFA_Node ownership
Currently a CXFA_Node can be owned by the CXFA_Document root pointer,
the CXFA_Document Purge list or as a child of another CXFA_Node. This
makes it hard to know who currently owns what.
This CL moves all ownership of nodes into a CXFA_NodeOwner class which
CXFA_Document subclasses. The node owner always owns all nodes and is
responsible for cleaning them up upon destruction. The destruction order
is not guarenteed to be in tree order as nodes can be inserted and moved
around the tree.
Change-Id: I6b202b8e844999ba835093dcdd1a808938b6d9a8
Reviewed-on: https://pdfium-review.googlesource.com/26434
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_nodeowner.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_nodeowner.cpp | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/xfa/fxfa/parser/cxfa_nodeowner.cpp b/xfa/fxfa/parser/cxfa_nodeowner.cpp new file mode 100644 index 0000000000..ae6f589616 --- /dev/null +++ b/xfa/fxfa/parser/cxfa_nodeowner.cpp @@ -0,0 +1,35 @@ +// Copyright 2018 PDFium Authors. All rights reserved. +// Use of this source code is governed by a BSD-style license that can be +// found in the LICENSE file. + +// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com + +#include "xfa/fxfa/parser/cxfa_nodeowner.h" + +#include <utility> + +#include "third_party/base/stl_util.h" +#include "xfa/fxfa/parser/cxfa_node.h" + +CXFA_NodeOwner::CXFA_NodeOwner() = default; + +CXFA_NodeOwner::~CXFA_NodeOwner() = default; + +CXFA_Node* CXFA_NodeOwner::AddOwnedNode(std::unique_ptr<CXFA_Node> node) { + if (!node) + return nullptr; + + CXFA_Node* ret = node.get(); + nodes_.insert(std::move(node)); + return ret; +} + +void CXFA_NodeOwner::FreeOwnedNode(CXFA_Node* node) { + if (!node) + return; + + pdfium::FakeUniquePtr<CXFA_Node> search(node); + auto it = nodes_.find(search); + assert(it != nodes_.end()); + nodes_.erase(it); +} |