diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-10 16:40:56 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-10 16:40:56 +0000 |
commit | 93cfa060aa6324b2a79f25782e3c986750479ac5 (patch) | |
tree | ca2ef7a14861235877b084f9433502b5507b36bb /xfa/fxfa/parser/cxfa_node.h | |
parent | 18a6069f5ce332b2bab97bf3a6d8ea9528d69791 (diff) | |
download | pdfium-93cfa060aa6324b2a79f25782e3c986750479ac5.tar.xz |
Remove 2 param CXFA_Node::GetNodeItem
This CL removes the CXFA_Node::GetNodeItem(kind, object_type) method in
favour of explicit methods. All of the callers passed in a
XFA_ObjectType::ContainerNode.
Change-Id: I713184d3fa188a122284f50ac2a6b0fd3d7aed26
Reviewed-on: https://pdfium-review.googlesource.com/22651
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_node.h')
-rw-r--r-- | xfa/fxfa/parser/cxfa_node.h | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.h b/xfa/fxfa/parser/cxfa_node.h index fe1c3e007f..c1074ffaad 100644 --- a/xfa/fxfa/parser/cxfa_node.h +++ b/xfa/fxfa/parser/cxfa_node.h @@ -55,13 +55,6 @@ enum XFA_NodeFlag { XFA_NodeFlag_LayoutGeneratedNode = 1 << 8 }; -enum XFA_NODEITEM { - XFA_NODEITEM_Parent, - XFA_NODEITEM_FirstChild, - XFA_NODEITEM_NextSibling, - XFA_NODEITEM_PrevSibling, -}; - class CXFA_Node : public CXFA_Object { public: struct PropertyData { @@ -188,7 +181,10 @@ class CXFA_Node : public CXFA_Object { CXFA_Node* GetFirstChild() const { return m_pChild; } CXFA_Node* GetParent() const { return m_pParent; } - CXFA_Node* GetNodeItem(XFA_NODEITEM eItem, XFA_ObjectType eType) const; + CXFA_Node* GetNextContainerSibling() const; + CXFA_Node* GetPrevContainerSibling() const; + CXFA_Node* GetFirstContainerChild() const; + CXFA_Node* GetContainerParent() const; std::vector<CXFA_Node*> GetNodeList(uint32_t dwTypeFilter, XFA_Element eTypeFilter); |