diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2018-04-09 16:06:24 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-09 16:06:24 +0000 |
commit | a0706f0c904673ea9f679829ff87e730e5800765 (patch) | |
tree | a803e20f27cc18a1daa5d6d7ef3244c1d160735c /xfa/fxfa/parser/cxfa_node.cpp | |
parent | 6058efdbdc186e120e7e2121c290ac4d820ffbf8 (diff) | |
download | pdfium-a0706f0c904673ea9f679829ff87e730e5800765.tar.xz |
Move code to set XML value to CXFA_Node
This CL moves the code to set the XML node data from CJX_Node to
CXFA_Node. The XML node is owned by the XFA node, not the CJX node so it
makes more sense to have the modifications happen there.
This combines the duplicate code in CJX_Node into a single SetToXML
method.
Bug: chromium:813155
Change-Id: I493725d1412688cb1a0d04bd9ae9fa5a36caebf3
Reviewed-on: https://pdfium-review.googlesource.com/29858
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_node.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index b2535aee8c..3823d2eae0 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -4656,3 +4656,38 @@ bool CXFA_Node::PresenceRequiresSpace() const { return ePresence == XFA_AttributeEnum::Visible || ePresence == XFA_AttributeEnum::Invisible; } + +void CXFA_Node::SetToXML(const WideString& value) { + auto* elem = static_cast<CFX_XMLElement*>(GetXMLMappingNode()); + FX_XMLNODETYPE eXMLType = elem->GetType(); + switch (eXMLType) { + case FX_XMLNODE_Element: { + if (IsAttributeInXML()) { + elem->SetString(JSObject()->GetCData(XFA_Attribute::QualifiedName), + value); + return; + } + + bool bDeleteChildren = true; + if (GetPacketType() == XFA_PacketType::Datasets) { + for (CXFA_Node* pChildDataNode = GetFirstChild(); pChildDataNode; + pChildDataNode = pChildDataNode->GetNextSibling()) { + if (!pChildDataNode->GetBindItems()->empty()) { + bDeleteChildren = false; + break; + } + } + } + if (bDeleteChildren) + elem->DeleteChildren(); + + elem->SetTextData(value); + break; + } + case FX_XMLNODE_Text: + static_cast<CFX_XMLText*>(GetXMLMappingNode())->SetText(value); + break; + default: + NOTREACHED(); + } +} |