summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--fxjs/cjx_node.cpp5
-rw-r--r--fxjs/cjx_node.h2
2 files changed, 3 insertions, 4 deletions
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp
index ba55086261..bf8c1f726e 100644
--- a/fxjs/cjx_node.cpp
+++ b/fxjs/cjx_node.cpp
@@ -3165,7 +3165,7 @@ bool CJX_Node::SetCData(XFA_Attribute eAttr,
return true;
}
-bool CJX_Node::SetAttributeValue(const WideString& wsValue,
+void CJX_Node::SetAttributeValue(const WideString& wsValue,
const WideString& wsXMLValue,
bool bNotify,
bool bScriptModify) {
@@ -3176,7 +3176,7 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue,
SetUserData(pKey, pClone, &deleteWideStringCallBack);
OnChanged(XFA_Attribute::Value, bNotify, bScriptModify);
if (!GetXFANode()->IsNeedSavingXMLNode())
- return true;
+ return;
auto* elem = static_cast<CFX_XMLElement*>(GetXFANode()->GetXMLMappingNode());
FX_XMLNODETYPE eXMLType = elem->GetType();
@@ -3211,7 +3211,6 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue,
default:
ASSERT(0);
}
- return true;
}
pdfium::Optional<WideString> CJX_Node::TryCData(XFA_Attribute eAttr,
diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h
index d0f8279e7f..148f1a6378 100644
--- a/fxjs/cjx_node.h
+++ b/fxjs/cjx_node.h
@@ -59,7 +59,7 @@ class CJX_Node : public CJX_Object {
pdfium::Optional<WideString> TryAttribute(XFA_Attribute eAttr,
bool bUseDefault);
- bool SetAttributeValue(const WideString& wsValue,
+ void SetAttributeValue(const WideString& wsValue,
const WideString& wsXMLValue,
bool bNotify,
bool bScriptModify);