diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-04 03:32:08 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-04 03:32:08 +0000 |
| commit | 5e35931f90d83e471e4d215034abb4dcb92fac3a (patch) | |
| tree | 6e33c19c2d637383d0e8b6ccd1632f66be30943d /xfa/fxfa/parser/cxfa_nodehelper.cpp | |
| parent | 185832ce3665ddfe7c3c96b398a4f56ce3eacf62 (diff) | |
| download | pdfium-5e35931f90d83e471e4d215034abb4dcb92fac3a.tar.xz | |
Remove default values from CXFA_Node::GetNodeList
This CL removes the default values and inlines in the call sites.
Change-Id: Iae95653a5d724918c5944f860a0743053e98fe88
Reviewed-on: https://pdfium-review.googlesource.com/17615
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_nodehelper.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_nodehelper.cpp | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/cxfa_nodehelper.cpp b/xfa/fxfa/parser/cxfa_nodehelper.cpp index 84237485b3..4c9e67b96c 100644 --- a/xfa/fxfa/parser/cxfa_nodehelper.cpp +++ b/xfa/fxfa/parser/cxfa_nodehelper.cpp @@ -69,7 +69,8 @@ int32_t CXFA_NodeHelper::NodeAcc_TraverseAnySiblings( return 0; int32_t nCount = 0; - for (CXFA_Node* child : parent->GetNodeList(XFA_NODEFILTER_Properties)) { + for (CXFA_Node* child : + parent->GetNodeList(XFA_NODEFILTER_Properties, XFA_Element::Unknown)) { if (bIsClassName) { if (child->GetClassHashCode() == dNameHash) { pSiblings->push_back(child); @@ -87,7 +88,8 @@ int32_t CXFA_NodeHelper::NodeAcc_TraverseAnySiblings( nCount += NodeAcc_TraverseAnySiblings(child, dNameHash, pSiblings, bIsClassName); } - for (CXFA_Node* child : parent->GetNodeList(XFA_NODEFILTER_Children)) { + for (CXFA_Node* child : + parent->GetNodeList(XFA_NODEFILTER_Children, XFA_Element::Unknown)) { if (bIsClassName) { if (child->GetClassHashCode() == dNameHash) { pSiblings->push_back(child); @@ -120,7 +122,8 @@ int32_t CXFA_NodeHelper::NodeAcc_TraverseSiblings( int32_t nCount = 0; if (bIsFindProperty) { - for (CXFA_Node* child : parent->GetNodeList(XFA_NODEFILTER_Properties)) { + for (CXFA_Node* child : + parent->GetNodeList(XFA_NODEFILTER_Properties, XFA_Element::Unknown)) { if (bIsClassName) { if (child->GetClassHashCode() == dNameHash) { pSiblings->push_back(child); @@ -145,7 +148,8 @@ int32_t CXFA_NodeHelper::NodeAcc_TraverseSiblings( if (nCount > 0) return nCount; } - for (CXFA_Node* child : parent->GetNodeList(XFA_NODEFILTER_Children)) { + for (CXFA_Node* child : + parent->GetNodeList(XFA_NODEFILTER_Children, XFA_Element::Unknown)) { if (child->GetElementType() == XFA_Element::Variables) continue; |