diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-20 20:47:28 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-20 20:47:28 +0000 |
| commit | ef2b74990f4f2a82e279ef5b8a911d89be1e83e2 (patch) | |
| tree | e9aa3e2a7178b38807a27b58292974d56cd446fa /xfa/fxfa/cxfa_ffdocview.cpp | |
| parent | 1b36b90f2e33f5a80e6ae9b22821a623a9d62782 (diff) | |
| download | pdfium-ef2b74990f4f2a82e279ef5b8a911d89be1e83e2.tar.xz | |
Convert CXFA_BindItemsData::Get* to return strings
This CL changes the CXFA_BindItemsData to return the WideString values
instead of taking out params.
Change-Id: Ic756da02121c4ce40a123b887cc774045cbfaa8e
Reviewed-on: https://pdfium-review.googlesource.com/18811
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffdocview.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_ffdocview.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/xfa/fxfa/cxfa_ffdocview.cpp b/xfa/fxfa/cxfa_ffdocview.cpp index 928bc1e0d4..e2a0adcc34 100644 --- a/xfa/fxfa/cxfa_ffdocview.cpp +++ b/xfa/fxfa/cxfa_ffdocview.cpp @@ -710,8 +710,7 @@ void CXFA_FFDocView::RunBindItems() { CXFA_BindItemsData bindItemsData(item); CFXJSE_Engine* pScriptContext = pWidgetNode->GetDocument()->GetScriptContext(); - WideString wsRef; - bindItemsData.GetRef(wsRef); + WideString wsRef = bindItemsData.GetRef(); uint32_t dwStyle = XFA_RESOLVENODE_Children | XFA_RESOLVENODE_Properties | XFA_RESOLVENODE_Siblings | XFA_RESOLVENODE_Parent | XFA_RESOLVENODE_ALL; @@ -722,10 +721,8 @@ void CXFA_FFDocView::RunBindItems() { if (rs.dwFlags != XFA_RESOLVENODE_RSTYPE_Nodes || rs.objects.empty()) continue; - WideString wsValueRef; - WideString wsLabelRef; - bindItemsData.GetValueRef(wsValueRef); - bindItemsData.GetLabelRef(wsLabelRef); + WideString wsValueRef = bindItemsData.GetValueRef(); + WideString wsLabelRef = bindItemsData.GetLabelRef(); const bool bUseValue = wsLabelRef.IsEmpty() || wsLabelRef == wsValueRef; const bool bLabelUseContent = wsLabelRef.IsEmpty() || wsLabelRef == L"$"; const bool bValueUseContent = wsValueRef.IsEmpty() || wsValueRef == L"$"; |