diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 16:44:21 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 16:44:21 +0000 |
commit | 72d718ffb0bb5320c94ca387cb4938d946b076fc (patch) | |
tree | 26202761b247a8ac7644b6d70bfd4764986ff0ed /xfa/fxfa/parser/cxfa_eventdata.cpp | |
parent | 000b95b3c9dd7c7fbfbd6359404a09dfdc2d5226 (diff) | |
download | pdfium-72d718ffb0bb5320c94ca387cb4938d946b076fc.tar.xz |
Cleanup CXFA_EventData methods
This CL changes the GetRef method to return a WideString instead of
taking an out param and removes the unused GetSignDataTarget method.
Change-Id: If50c4243fa136000fda8e4411ada4cd9cd8b20d5
Reviewed-on: https://pdfium-review.googlesource.com/18911
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_eventdata.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_eventdata.cpp | 13 |
1 files changed, 2 insertions, 11 deletions
diff --git a/xfa/fxfa/parser/cxfa_eventdata.cpp b/xfa/fxfa/parser/cxfa_eventdata.cpp index b9b3c8c0c2..1485d8e0f5 100644 --- a/xfa/fxfa/parser/cxfa_eventdata.cpp +++ b/xfa/fxfa/parser/cxfa_eventdata.cpp @@ -26,8 +26,8 @@ XFA_Element CXFA_EventData::GetEventType() const { return XFA_Element::Unknown; } -void CXFA_EventData::GetRef(WideString& wsRef) { - wsRef = m_pNode->JSNode()->GetCData(XFA_Attribute::Ref); +WideString CXFA_EventData::GetRef() const { + return m_pNode->JSNode()->GetCData(XFA_Attribute::Ref); } CXFA_ScriptData CXFA_EventData::GetScriptData() const { @@ -37,12 +37,3 @@ CXFA_ScriptData CXFA_EventData::GetScriptData() const { CXFA_SubmitData CXFA_EventData::GetSubmitData() const { return CXFA_SubmitData(m_pNode->GetChild(0, XFA_Element::Submit, false)); } - -void CXFA_EventData::GetSignDataTarget(WideString& wsTarget) { - CXFA_Node* pNode = - m_pNode->JSNode()->GetProperty(0, XFA_Element::SignData, true); - if (!pNode) - return; - - wsTarget = pNode->JSNode()->GetCData(XFA_Attribute::Target); -} |