diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 22:05:40 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 22:05:40 +0000 |
| commit | ebe865db548f768038186054fc0f1eb024565c43 (patch) | |
| tree | 582743c8c367a39a2471aa238787e91c26bc4e92 /xfa | |
| parent | 559f974dbb92e35a1bfb739032d82a2536fcf59c (diff) | |
| download | pdfium-ebe865db548f768038186054fc0f1eb024565c43.tar.xz | |
Cleanup CXFA_SubmitData
This CL cleans up the CXFA_SubmitData methods by using returns instead
of out-params, sets the correct return types and makes the methods
const.
Change-Id: I41fe8f59e450ca444b64c291ac4a0a199c7ad0cf
Reviewed-on: https://pdfium-review.googlesource.com/19111
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_submitdata.cpp | 12 | ||||
| -rw-r--r-- | xfa/fxfa/parser/cxfa_submitdata.h | 8 |
2 files changed, 10 insertions, 10 deletions
diff --git a/xfa/fxfa/parser/cxfa_submitdata.cpp b/xfa/fxfa/parser/cxfa_submitdata.cpp index c461ed2a09..e10dec62f0 100644 --- a/xfa/fxfa/parser/cxfa_submitdata.cpp +++ b/xfa/fxfa/parser/cxfa_submitdata.cpp @@ -10,18 +10,18 @@ CXFA_SubmitData::CXFA_SubmitData(CXFA_Node* pNode) : CXFA_DataData(pNode) {} -bool CXFA_SubmitData::IsSubmitEmbedPDF() { +bool CXFA_SubmitData::IsSubmitEmbedPDF() const { return m_pNode->JSNode()->GetBoolean(XFA_Attribute::EmbedPDF); } -int32_t CXFA_SubmitData::GetSubmitFormat() { +XFA_ATTRIBUTEENUM CXFA_SubmitData::GetSubmitFormat() const { return m_pNode->JSNode()->GetEnum(XFA_Attribute::Format); } -void CXFA_SubmitData::GetSubmitTarget(WideString& wsTarget) { - wsTarget = m_pNode->JSNode()->GetCData(XFA_Attribute::Target); +WideString CXFA_SubmitData::GetSubmitTarget() const { + return m_pNode->JSNode()->GetCData(XFA_Attribute::Target); } -void CXFA_SubmitData::GetSubmitXDPContent(WideString& wsContent) { - wsContent = m_pNode->JSNode()->GetCData(XFA_Attribute::XdpContent); +WideString CXFA_SubmitData::GetSubmitXDPContent() const { + return m_pNode->JSNode()->GetCData(XFA_Attribute::XdpContent); } diff --git a/xfa/fxfa/parser/cxfa_submitdata.h b/xfa/fxfa/parser/cxfa_submitdata.h index e90848d9f1..039c327b97 100644 --- a/xfa/fxfa/parser/cxfa_submitdata.h +++ b/xfa/fxfa/parser/cxfa_submitdata.h @@ -17,10 +17,10 @@ class CXFA_SubmitData : public CXFA_DataData { public: explicit CXFA_SubmitData(CXFA_Node* pNode); - bool IsSubmitEmbedPDF(); - int32_t GetSubmitFormat(); - void GetSubmitTarget(WideString& wsTarget); - void GetSubmitXDPContent(WideString& wsContent); + bool IsSubmitEmbedPDF() const; + XFA_ATTRIBUTEENUM GetSubmitFormat() const; + WideString GetSubmitTarget() const; + WideString GetSubmitXDPContent() const; }; #endif // XFA_FXFA_PARSER_CXFA_SUBMITDATA_H_ |