diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-01-10 16:30:56 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-01-10 16:30:56 +0000 |
| commit | 18a6069f5ce332b2bab97bf3a6d8ea9528d69791 (patch) | |
| tree | 3ff25aea65bc9691b05919cec0dc74a6a506d156 /xfa/fxfa/parser/cxfa_nodelocale.cpp | |
| parent | 640d8ffad8536c789103892c7a4e69e5d30172c8 (diff) | |
| download | pdfium-18a6069f5ce332b2bab97bf3a6d8ea9528d69791.tar.xz | |
Remove CXFA_Node::GetNodeItem
This CL removes the generic CXFA_Node::GetNodeItem(type) method for
specific calls to get the parent, child or siblings.
Change-Id: Ief68284ac7b954aaa4ed7120dd82a39f8b52656a
Reviewed-on: https://pdfium-review.googlesource.com/22650
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_nodelocale.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_nodelocale.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/xfa/fxfa/parser/cxfa_nodelocale.cpp b/xfa/fxfa/parser/cxfa_nodelocale.cpp index 5031216de9..92a676a762 100644 --- a/xfa/fxfa/parser/cxfa_nodelocale.cpp +++ b/xfa/fxfa/parser/cxfa_nodelocale.cpp @@ -136,13 +136,12 @@ WideString CXFA_NodeLocale::GetNumPattern(FX_LOCALENUMSUBCATEGORY eType) const { CXFA_Node* CXFA_NodeLocale::GetNodeByName(CXFA_Node* pParent, const WideStringView& wsName) const { - CXFA_Node* pChild = - pParent ? pParent->GetNodeItem(XFA_NODEITEM_FirstChild) : nullptr; + CXFA_Node* pChild = pParent ? pParent->GetFirstChild() : nullptr; while (pChild) { if (pChild->JSObject()->GetAttribute(XFA_Attribute::Name) == wsName) return pChild; - pChild = pChild->GetNodeItem(XFA_NODEITEM_NextSibling); + pChild = pChild->GetNextSibling(); } return nullptr; } |