summaryrefslogtreecommitdiff
path: root/fxjs
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-11-16 13:42:49 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-11-16 13:42:49 +0000
commitb066704a22ba4f242567f508c12bf2545cbed9e1 (patch)
treeb0ef12e2873bf7018d4b17a41b626428fb789923 /fxjs
parent4011677aed8b258fcf87cf52b0d541ef04c832ff (diff)
downloadpdfium-b066704a22ba4f242567f508c12bf2545cbed9e1.tar.xz
Convert TryCData and TryContent to optionals
This CL changes the TryCData and TryContent to return pdfium::Optional<WideString> values instead of returning a bool and taking an out WideString. Change-Id: I9c9d877803f9f1977191e12d6a907c29784c10b2 Reviewed-on: https://pdfium-review.googlesource.com/18510 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs')
-rw-r--r--fxjs/cfxjse_engine.cpp7
-rw-r--r--fxjs/cjx_node.cpp61
-rw-r--r--fxjs/cjx_node.h5
3 files changed, 32 insertions, 41 deletions
diff --git a/fxjs/cfxjse_engine.cpp b/fxjs/cfxjse_engine.cpp
index eb76ef1c57..e5d6c0a0fe 100644
--- a/fxjs/cfxjse_engine.cpp
+++ b/fxjs/cfxjse_engine.cpp
@@ -501,11 +501,12 @@ bool CFXJSE_Engine::RunVariablesScript(CXFA_Node* pScriptNode) {
if (!pTextNode)
return false;
- WideString wsScript;
- if (!pTextNode->JSNode()->TryCData(XFA_Attribute::Value, wsScript, true))
+ pdfium::Optional<WideString> wsScript =
+ pTextNode->JSNode()->TryCData(XFA_Attribute::Value, true);
+ if (!wsScript)
return false;
- ByteString btScript = wsScript.UTF8Encode();
+ ByteString btScript = wsScript->UTF8Encode();
auto hRetValue = pdfium::MakeUnique<CFXJSE_Value>(m_pIsolate);
CXFA_Node* pThisObject = pParent->GetNodeItem(XFA_NODEITEM_Parent);
CFXJSE_Context* pVariablesContext =
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp
index 0af65464c8..a1b2f048dc 100644
--- a/fxjs/cjx_node.cpp
+++ b/fxjs/cjx_node.cpp
@@ -278,7 +278,12 @@ bool CJX_Node::GetAttribute(XFA_Attribute eAttr,
return true;
}
case XFA_AttributeType::CData: {
- return TryCData(pAttr->eName, wsValue, bUseDefault);
+ pdfium::Optional<WideString> ret = TryCData(pAttr->eName, bUseDefault);
+ if (!ret)
+ return false;
+
+ wsValue = *ret;
+ return true;
}
case XFA_AttributeType::Boolean: {
bool bValue;
@@ -964,9 +969,7 @@ void CJX_Node::Script_NodeClass_IsNull(CFXJSE_Value* pValue,
pValue->SetBoolean(false);
return;
}
-
- WideString strValue;
- pValue->SetBoolean(!TryContent(strValue, false, true) || strValue.IsEmpty());
+ pValue->SetBoolean(GetContent(false).IsEmpty());
}
void CJX_Node::Script_NodeClass_OneOfChild(CFXJSE_Value* pValue,
@@ -3128,8 +3131,7 @@ CXFA_Measurement CJX_Node::GetMeasure(XFA_Attribute eAttr) const {
}
WideString CJX_Node::GetCData(XFA_Attribute eAttr) {
- WideString wsValue;
- return TryCData(eAttr, wsValue, true) ? wsValue : WideString();
+ return TryCData(eAttr, true).value_or(WideString());
}
bool CJX_Node::SetCData(XFA_Attribute eAttr,
@@ -3259,30 +3261,22 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue,
return true;
}
-bool CJX_Node::TryCData(XFA_Attribute eAttr,
- WideString& wsValue,
- bool bUseDefault) {
+pdfium::Optional<WideString> CJX_Node::TryCData(XFA_Attribute eAttr,
+ bool bUseDefault) {
void* pKey = GetMapKey_Element(GetXFANode()->GetElementType(), eAttr);
if (eAttr == XFA_Attribute::Value) {
WideString* pStr = (WideString*)GetUserData(pKey, true);
- if (pStr) {
- wsValue = *pStr;
- return true;
- }
+ if (pStr)
+ return {*pStr};
} else {
WideStringView wsValueC;
- if (GetMapModuleString(pKey, wsValueC)) {
- wsValue = wsValueC;
- return true;
- }
+ if (GetMapModuleString(pKey, wsValueC))
+ return {WideString(wsValueC)};
}
if (!bUseDefault)
- return false;
+ return {};
- pdfium::Optional<WideString> str = GetXFANode()->GetDefaultCData(eAttr);
- if (str)
- wsValue = *str;
- return !!str;
+ return GetXFANode()->GetDefaultCData(eAttr);
}
bool CJX_Node::SetObject(XFA_Attribute eAttr,
@@ -3544,13 +3538,11 @@ bool CJX_Node::SetContent(const WideString& wsContent,
}
WideString CJX_Node::GetContent(bool bScriptModify) {
- WideString wsContent;
- return TryContent(wsContent, bScriptModify, true) ? wsContent : WideString();
+ return TryContent(bScriptModify, true).value_or(WideString());
}
-bool CJX_Node::TryContent(WideString& wsContent,
- bool bScriptModify,
- bool bProto) {
+pdfium::Optional<WideString> CJX_Node::TryContent(bool bScriptModify,
+ bool bProto) {
CXFA_Node* pNode = nullptr;
switch (GetXFANode()->GetObjectType()) {
case XFA_ObjectType::ContainerNode:
@@ -3560,16 +3552,16 @@ bool CJX_Node::TryContent(WideString& wsContent,
CXFA_Node* pValue =
GetXFANode()->GetChild(0, XFA_Element::Value, false);
if (!pValue)
- return false;
+ return {};
CXFA_Node* pChildValue = pValue->GetNodeItem(XFA_NODEITEM_FirstChild);
if (pChildValue && XFA_FieldIsMultiListBox(GetXFANode())) {
pChildValue->JSNode()->SetAttribute(XFA_Attribute::ContentType,
L"text/xml", false);
}
- return pChildValue ? pChildValue->JSNode()->TryContent(
- wsContent, bScriptModify, bProto)
- : false;
+ if (pChildValue)
+ return pChildValue->JSNode()->TryContent(bScriptModify, bProto);
+ return {};
}
break;
case XFA_ObjectType::ContentNode: {
@@ -3588,8 +3580,7 @@ bool CJX_Node::TryContent(WideString& wsContent,
pContentRawDataNode = GetXFANode()->CreateSamePacketNode(element);
GetXFANode()->InsertChild(pContentRawDataNode, nullptr);
}
- return pContentRawDataNode->JSNode()->TryContent(wsContent, bScriptModify,
- true);
+ return pContentRawDataNode->JSNode()->TryContent(bScriptModify, true);
}
case XFA_ObjectType::NodeC:
case XFA_ObjectType::NodeV:
@@ -3606,9 +3597,9 @@ bool CJX_Node::TryContent(WideString& wsContent,
if (pScriptContext)
GetDocument()->GetScriptContext()->AddNodesOfRunScript(GetXFANode());
}
- return TryCData(XFA_Attribute::Value, wsContent, false);
+ return TryCData(XFA_Attribute::Value, false);
}
- return false;
+ return {};
}
bool CJX_Node::TryNamespace(WideString& wsNamespace) {
diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h
index fa2b28e5e2..2e9360d451 100644
--- a/fxjs/cjx_node.h
+++ b/fxjs/cjx_node.h
@@ -61,6 +61,7 @@ class CJX_Node : public CJX_Object {
XFA_Element eType,
bool bCreateProperty);
+ pdfium::Optional<WideString> TryContent(bool bScriptModify, bool bProto);
bool SetContent(const WideString& wsContent,
const WideString& wsXMLValue,
bool bNotify,
@@ -72,15 +73,13 @@ class CJX_Node : public CJX_Object {
bool SetInteger(XFA_Attribute eAttr, int32_t iValue, bool bNotify);
int32_t GetInteger(XFA_Attribute eAttr);
- bool TryCData(XFA_Attribute eAttr, WideString& wsValue, bool bUseDefault);
+ pdfium::Optional<WideString> TryCData(XFA_Attribute eAttr, bool bUseDefault);
bool SetCData(XFA_Attribute eAttr,
const WideString& wsValue,
bool bNotify,
bool bScriptModify);
WideString GetCData(XFA_Attribute eAttr);
- bool TryContent(WideString& wsContent, bool bScriptModify, bool bProto);
-
bool TryEnum(XFA_Attribute eAttr,
XFA_ATTRIBUTEENUM& eValue,
bool bUseDefault);