diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-04 01:49:37 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-04 01:49:37 +0000 |
commit | c5d5729700a4d145cd298ec52f6f57b2d11aaafe (patch) | |
tree | b9929296fc45c754d5f7748613cec15020664052 /xfa/fxfa/cxfa_ffdocview.cpp | |
parent | cf705715c5311057933ac03b395d9ea75b6f0660 (diff) | |
download | pdfium-c5d5729700a4d145cd298ec52f6f57b2d11aaafe.tar.xz |
Remove default values from CXFA_Node::{CountChildren|GetChild}chromium/3259
This CL removes the default values and inlines into the call sites as
needed.
Change-Id: I554e9d5ac781003695f90522f9c6c3160653eb59
Reviewed-on: https://pdfium-review.googlesource.com/17572
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffdocview.cpp')
-rw-r--r-- | xfa/fxfa/cxfa_ffdocview.cpp | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/xfa/fxfa/cxfa_ffdocview.cpp b/xfa/fxfa/cxfa_ffdocview.cpp index a2c4893446..cb2155d374 100644 --- a/xfa/fxfa/cxfa_ffdocview.cpp +++ b/xfa/fxfa/cxfa_ffdocview.cpp @@ -113,7 +113,7 @@ void CXFA_FFDocView::StopLayout() { if (!pRootItem) return; - CXFA_Node* pSubformNode = pRootItem->GetChild(0, XFA_Element::Subform); + CXFA_Node* pSubformNode = pRootItem->GetChild(0, XFA_Element::Subform, false); if (!pSubformNode) return; @@ -269,16 +269,17 @@ int32_t CXFA_FFDocView::ProcessWidgetEvent(CXFA_EventParam* pParam, ToNode(m_pDoc->GetXFADoc()->GetXFAObject(XFA_HASHCODE_Config)); if (pConfigItem) { CXFA_Node* pValidateNode = nullptr; - CXFA_Node* pAcrobatNode = pConfigItem->GetChild(0, XFA_Element::Acrobat); - pValidateNode = pAcrobatNode - ? pAcrobatNode->GetChild(0, XFA_Element::Validate) - : nullptr; + CXFA_Node* pAcrobatNode = + pConfigItem->GetChild(0, XFA_Element::Acrobat, false); + pValidateNode = + pAcrobatNode ? pAcrobatNode->GetChild(0, XFA_Element::Validate, false) + : nullptr; if (!pValidateNode) { CXFA_Node* pPresentNode = - pConfigItem->GetChild(0, XFA_Element::Present); - pValidateNode = pPresentNode - ? pPresentNode->GetChild(0, XFA_Element::Validate) - : nullptr; + pConfigItem->GetChild(0, XFA_Element::Present, false); + pValidateNode = pPresentNode ? pPresentNode->GetChild( + 0, XFA_Element::Validate, false) + : nullptr; } if (pValidateNode) wsValidateStr = pValidateNode->JSNode()->GetContent(false); @@ -295,7 +296,7 @@ int32_t CXFA_FFDocView::ProcessWidgetEvent(CXFA_EventParam* pParam, if (!pRootItem) return XFA_EVENTERROR_Error; - pNode = pRootItem->GetChild(0, XFA_Element::Subform); + pNode = pRootItem->GetChild(0, XFA_Element::Subform, false); } ExecEventActivityByDeepFirst(pNode, pParam->m_eType, pParam->m_bIsFormReady, |