diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 21:52:50 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 21:52:50 +0000 |
commit | 64e16079bb87085fdff5d6268267fd01eacbc5d9 (patch) | |
tree | 0c13b2e9fb686dc18cdf49f4091e22f089eea65c /xfa/fxfa/cxfa_widgetacc.cpp | |
parent | f9958ed00c5f288d95ab4c6c1d90fc639d5c0f8d (diff) | |
download | pdfium-64e16079bb87085fdff5d6268267fd01eacbc5d9.tar.xz |
Cleanup CXFA_ImageData
This CL removes the second constructor parameter which is always true
and removes all related conditionals. The methods have been changed to
const and return instead of using out parameters.
Change-Id: Id6d2aa533a212e961e3b8169eef57b7e868d503e
Reviewed-on: https://pdfium-review.googlesource.com/19010
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_widgetacc.cpp')
-rw-r--r-- | xfa/fxfa/cxfa_widgetacc.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/cxfa_widgetacc.cpp b/xfa/fxfa/cxfa_widgetacc.cpp index 5bff539366..26ba231533 100644 --- a/xfa/fxfa/cxfa_widgetacc.cpp +++ b/xfa/fxfa/cxfa_widgetacc.cpp @@ -176,9 +176,9 @@ void CXFA_WidgetAcc::ResetData() { CXFA_ImageData imageData = imageValueData.GetImageData(); WideString wsContentType, wsHref; if (imageData.HasValidNode()) { - imageData.GetContent(wsValue); - imageData.GetContentType(wsContentType); - imageData.GetHref(wsHref); + wsValue = imageData.GetContent(); + wsContentType = imageData.GetContentType(); + wsHref = imageData.GetHref(); } SetImageEdit(wsContentType, wsHref, wsValue); break; |