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/cxfa_ffdocview.cpp | |
| 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/cxfa_ffdocview.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffdocview.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fxfa/cxfa_ffdocview.cpp b/xfa/fxfa/cxfa_ffdocview.cpp index 681d4cd8f8..52b7f64cf7 100644 --- a/xfa/fxfa/cxfa_ffdocview.cpp +++ b/xfa/fxfa/cxfa_ffdocview.cpp @@ -467,10 +467,8 @@ int32_t CXFA_FFDocView::ExecEventActivityByDeepFirst(CXFA_Node* pFormNode, int32_t iRet = XFA_EVENTERROR_NotExist; if (bRecursive) { - for (CXFA_Node* pNode = pFormNode->GetNodeItem( - XFA_NODEITEM_FirstChild, XFA_ObjectType::ContainerNode); - pNode; pNode = pNode->GetNodeItem(XFA_NODEITEM_NextSibling, - XFA_ObjectType::ContainerNode)) { + for (CXFA_Node* pNode = pFormNode->GetFirstContainerChild(); pNode; + pNode = pNode->GetNextContainerSibling()) { elementType = pNode->GetElementType(); if (elementType != XFA_Element::Variables && elementType != XFA_Element::Draw) { |