diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-04-05 18:33:35 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-05 18:33:35 +0000 |
commit | 72fe14aa5f56ac68be0842c3b0f2e799f145bed3 (patch) | |
tree | 55ce307785ed4c87a9b5e229c106012c00ffcf6a | |
parent | 4a979709304592be8511458793463aa2570bd7af (diff) | |
download | pdfium-72fe14aa5f56ac68be0842c3b0f2e799f145bed3.tar.xz |
Remove unused param.
This CL removes the unused CXFA_Document param from various anonymous
namespace methods in CXFA_Document.
Change-Id: Ic10b4d2830936091da4db0919b33c16875cbd427
Reviewed-on: https://pdfium-review.googlesource.com/29853
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
-rw-r--r-- | xfa/fxfa/parser/cxfa_document.cpp | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/xfa/fxfa/parser/cxfa_document.cpp b/xfa/fxfa/parser/cxfa_document.cpp index 1160a8ad2e..2750157717 100644 --- a/xfa/fxfa/parser/cxfa_document.cpp +++ b/xfa/fxfa/parser/cxfa_document.cpp @@ -136,9 +136,7 @@ CXFA_Node* GetGlobalBinding(CXFA_Document* pDocument, uint32_t dwNameHash) { return it != pDocument->m_rgGlobalBinding.end() ? it->second : nullptr; } -void MergeNodeRecurse(CXFA_Document* pDocument, - CXFA_Node* pDestNodeParent, - CXFA_Node* pProtoNode) { +void MergeNodeRecurse(CXFA_Node* pDestNodeParent, CXFA_Node* pProtoNode) { CXFA_Node* pExistingNode = nullptr; for (CXFA_Node* pFormChild = pDestNodeParent->GetFirstChild(); pFormChild; pFormChild = pFormChild->GetNextSibling()) { @@ -155,7 +153,7 @@ void MergeNodeRecurse(CXFA_Document* pDocument, pExistingNode->SetTemplateNode(pProtoNode); for (CXFA_Node* pTemplateChild = pProtoNode->GetFirstChild(); pTemplateChild; pTemplateChild = pTemplateChild->GetNextSibling()) { - MergeNodeRecurse(pDocument, pExistingNode, pTemplateChild); + MergeNodeRecurse(pExistingNode, pTemplateChild); } return; } @@ -164,9 +162,7 @@ void MergeNodeRecurse(CXFA_Document* pDocument, pDestNodeParent->InsertChild(pNewNode, nullptr); } -void MergeNode(CXFA_Document* pDocument, - CXFA_Node* pDestNode, - CXFA_Node* pProtoNode) { +void MergeNode(CXFA_Node* pDestNode, CXFA_Node* pProtoNode) { { CXFA_NodeIterator sIterator(pDestNode); for (CXFA_Node* pNode = sIterator.GetCurrent(); pNode; @@ -177,7 +173,7 @@ void MergeNode(CXFA_Document* pDocument, pDestNode->SetTemplateNode(pProtoNode); for (CXFA_Node* pTemplateChild = pProtoNode->GetFirstChild(); pTemplateChild; pTemplateChild = pTemplateChild->GetNextSibling()) { - MergeNodeRecurse(pDocument, pDestNode, pTemplateChild); + MergeNodeRecurse(pDestNode, pTemplateChild); } { CXFA_NodeIterator sIterator(pDestNode); @@ -326,8 +322,7 @@ CXFA_Node* FindGlobalDataNode(CXFA_Document* pDocument, return pBounded; } -CXFA_Node* FindOnceDataNode(CXFA_Document* pDocument, - const WideString& wsName, +CXFA_Node* FindOnceDataNode(const WideString& wsName, CXFA_Node* pDataScope, XFA_Element eMatchNodeType) { if (wsName.IsEmpty()) @@ -463,7 +458,6 @@ CXFA_Node* FindMatchingDataNode( case XFA_AttributeEnum::Once: { bAccessedDataDOM = true; CXFA_Node* pOnceBindNode = FindOnceDataNode( - pDocument, pCurTemplateNode->JSObject()->GetCData(XFA_Attribute::Name), pDataScope, eMatchNodeType); if (!pOnceBindNode) { @@ -1584,7 +1578,7 @@ void CXFA_Document::DoProtoMerge() { if (!pProtoNode) continue; - MergeNode(this, pUseHrefNode, pProtoNode); + MergeNode(pUseHrefNode, pProtoNode); } } |