diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-04 03:31:29 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-04 03:31:29 +0000 |
| commit | 185832ce3665ddfe7c3c96b398a4f56ce3eacf62 (patch) | |
| tree | 98d3bce032a949f5ef79b15b7545f5ed25d16af5 /xfa/fxfa/parser/cxfa_layoutpagemgr.cpp | |
| parent | 33591752d2cb14f2e07726ca52afce6efbdc07c9 (diff) | |
| download | pdfium-185832ce3665ddfe7c3c96b398a4f56ce3eacf62.tar.xz | |
Remove default params for CXFA_Node::{Insert|Remove}Child
This CL removes the default values and inlines in the call sites as
needed.
Change-Id: I68925d57b6230400429421ce728dd5524e5fc9a1
Reviewed-on: https://pdfium-review.googlesource.com/17710
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_layoutpagemgr.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_layoutpagemgr.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/parser/cxfa_layoutpagemgr.cpp b/xfa/fxfa/parser/cxfa_layoutpagemgr.cpp index 1b72331608..3711752746 100644 --- a/xfa/fxfa/parser/cxfa_layoutpagemgr.cpp +++ b/xfa/fxfa/parser/cxfa_layoutpagemgr.cpp @@ -1791,7 +1791,7 @@ void CXFA_LayoutPageMgr::MergePageSetContents() { } } if (pExistingNode) { - pParentNode->RemoveChild(pExistingNode); + pParentNode->RemoveChild(pExistingNode, true); } } pContainerItem->m_pOldSubform = pNewSubform; @@ -1825,7 +1825,7 @@ void CXFA_LayoutPageMgr::MergePageSetContents() { pDocument->GetXFAObject(XFA_HASHCODE_Form) ->AsNode() ->GetFirstChildByClass(XFA_Element::Subform); - pFormToplevelSubform->InsertChild(pPendingPageSet); + pFormToplevelSubform->InsertChild(pPendingPageSet, nullptr); } pDocument->DataMerge_UpdateBindingRelations(pPendingPageSet); pPendingPageSet->SetFlag(XFA_NodeFlag_Initialized, true); @@ -1862,7 +1862,7 @@ void CXFA_LayoutPageMgr::MergePageSetContents() { } } CXFA_Node* pNext = sIterator.SkipChildrenAndMoveToNext(); - pNode->GetNodeItem(XFA_NODEITEM_Parent)->RemoveChild(pNode); + pNode->GetNodeItem(XFA_NODEITEM_Parent)->RemoveChild(pNode, true); pNode = pNext; } else { pNode->ClearFlag(XFA_NodeFlag_UnusedNode); |