summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2018-04-05 18:33:35 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-04-05 18:33:35 +0000
commit72fe14aa5f56ac68be0842c3b0f2e799f145bed3 (patch)
tree55ce307785ed4c87a9b5e229c106012c00ffcf6a
parent4a979709304592be8511458793463aa2570bd7af (diff)
downloadpdfium-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.cpp18
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);
}
}