summaryrefslogtreecommitdiff
path: root/xfa/fxfa/parser
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2017-11-22 22:09:42 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-11-22 22:09:42 +0000
commitfd16e44aa9b0e63e6d25254a3b812fb27152b6f4 (patch)
tree92092b4cc86a05f90518b63c4f884566380bfd92 /xfa/fxfa/parser
parent9d0d36b7f3e2106bbeb332edf821fc2c637e6ea0 (diff)
downloadpdfium-fd16e44aa9b0e63e6d25254a3b812fb27152b6f4.tar.xz
Cleanup CXFA_ValueData
This CL makes the methods const and uses return values instead of out-params. Change-Id: If760cd8c10d8f3a94dd3e13c46200cb6738a4d5a Reviewed-on: https://pdfium-review.googlesource.com/19150 Commit-Queue: dsinclair <dsinclair@chromium.org> Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser')
-rw-r--r--xfa/fxfa/parser/cxfa_valuedata.cpp28
-rw-r--r--xfa/fxfa/parser/cxfa_valuedata.h16
-rw-r--r--xfa/fxfa/parser/xfa_document_datamerger_imp.cpp10
3 files changed, 24 insertions, 30 deletions
diff --git a/xfa/fxfa/parser/cxfa_valuedata.cpp b/xfa/fxfa/parser/cxfa_valuedata.cpp
index 1ca051e637..d0f86a8b23 100644
--- a/xfa/fxfa/parser/cxfa_valuedata.cpp
+++ b/xfa/fxfa/parser/cxfa_valuedata.cpp
@@ -8,7 +8,7 @@
#include "xfa/fxfa/parser/cxfa_node.h"
-XFA_Element CXFA_ValueData::GetChildValueClassID() {
+XFA_Element CXFA_ValueData::GetChildValueClassID() const {
if (!m_pNode)
return XFA_Element::Unknown;
if (CXFA_Node* pNode = m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild))
@@ -16,49 +16,43 @@ XFA_Element CXFA_ValueData::GetChildValueClassID() {
return XFA_Element::Unknown;
}
-bool CXFA_ValueData::GetChildValueContent(WideString& wsContent) {
+WideString CXFA_ValueData::GetChildValueContent() const {
if (!m_pNode)
- return false;
+ return L"";
CXFA_Node* pNode = m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild);
if (!pNode)
- return false;
+ return L"";
- pdfium::Optional<WideString> content =
- pNode->JSNode()->TryContent(false, true);
- if (!content)
- return false;
-
- wsContent = *content;
- return true;
+ return pNode->JSNode()->TryContent(false, true).value_or(L"");
}
-CXFA_ArcData CXFA_ValueData::GetArcData() {
+CXFA_ArcData CXFA_ValueData::GetArcData() const {
return CXFA_ArcData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)
: nullptr);
}
-CXFA_LineData CXFA_ValueData::GetLineData() {
+CXFA_LineData CXFA_ValueData::GetLineData() const {
return CXFA_LineData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)
: nullptr);
}
-CXFA_RectangleData CXFA_ValueData::GetRectangleData() {
+CXFA_RectangleData CXFA_ValueData::GetRectangleData() const {
return CXFA_RectangleData(
m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild) : nullptr);
}
-CXFA_TextData CXFA_ValueData::GetTextData() {
+CXFA_TextData CXFA_ValueData::GetTextData() const {
return CXFA_TextData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)
: nullptr);
}
-CXFA_ExDataData CXFA_ValueData::GetExData() {
+CXFA_ExDataData CXFA_ValueData::GetExData() const {
return CXFA_ExDataData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)
: nullptr);
}
-CXFA_ImageData CXFA_ValueData::GetImageData() {
+CXFA_ImageData CXFA_ValueData::GetImageData() const {
return CXFA_ImageData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)
: nullptr);
}
diff --git a/xfa/fxfa/parser/cxfa_valuedata.h b/xfa/fxfa/parser/cxfa_valuedata.h
index 7e647d8d2a..101c960325 100644
--- a/xfa/fxfa/parser/cxfa_valuedata.h
+++ b/xfa/fxfa/parser/cxfa_valuedata.h
@@ -22,14 +22,14 @@ class CXFA_ValueData : public CXFA_DataData {
public:
explicit CXFA_ValueData(CXFA_Node* pNode) : CXFA_DataData(pNode) {}
- XFA_Element GetChildValueClassID();
- bool GetChildValueContent(WideString& wsContent);
- CXFA_ArcData GetArcData();
- CXFA_LineData GetLineData();
- CXFA_RectangleData GetRectangleData();
- CXFA_TextData GetTextData();
- CXFA_ExDataData GetExData();
- CXFA_ImageData GetImageData();
+ XFA_Element GetChildValueClassID() const;
+ WideString GetChildValueContent() const;
+ CXFA_ArcData GetArcData() const;
+ CXFA_LineData GetLineData() const;
+ CXFA_RectangleData GetRectangleData() const;
+ CXFA_TextData GetTextData() const;
+ CXFA_ExDataData GetExData() const;
+ CXFA_ImageData GetImageData() const;
};
#endif // XFA_FXFA_PARSER_CXFA_VALUEDATA_H_
diff --git a/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp b/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp
index 381e176927..2a3494f276 100644
--- a/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp
+++ b/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp
@@ -152,7 +152,7 @@ void CreateDataBinding(CXFA_Node* pFormNode,
break;
}
case XFA_Element::ChoiceList:
- defValueData.GetChildValueContent(wsValue);
+ wsValue = defValueData.GetChildValueContent();
if (pWidgetData->GetChoiceListOpen() == XFA_ATTRIBUTEENUM_MultiSelect) {
std::vector<WideString> wsSelTextArray =
pWidgetData->GetSelectedItemsValue();
@@ -180,7 +180,7 @@ void CreateDataBinding(CXFA_Node* pFormNode,
}
break;
case XFA_Element::CheckButton:
- defValueData.GetChildValueContent(wsValue);
+ wsValue = defValueData.GetChildValueContent();
if (wsValue.IsEmpty())
break;
@@ -199,7 +199,7 @@ void CreateDataBinding(CXFA_Node* pFormNode,
if (!pValue)
continue;
- CXFA_ValueData(pValue).GetChildValueContent(wsValue);
+ wsValue = CXFA_ValueData(pValue).GetChildValueContent();
if (wsValue.IsEmpty())
continue;
@@ -249,7 +249,7 @@ void CreateDataBinding(CXFA_Node* pFormNode,
break;
}
case XFA_Element::NumericEdit: {
- defValueData.GetChildValueContent(wsValue);
+ wsValue = defValueData.GetChildValueContent();
if (wsValue.IsEmpty())
break;
@@ -265,7 +265,7 @@ void CreateDataBinding(CXFA_Node* pFormNode,
break;
}
default:
- defValueData.GetChildValueContent(wsValue);
+ wsValue = defValueData.GetChildValueContent();
if (wsValue.IsEmpty())
break;